Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix constructor ambiguity error llvm::APInt in tests LLVM 11 #138

Merged
merged 1 commit into from
May 26, 2021

Conversation

ekilmer
Copy link
Contributor

@ekilmer ekilmer commented May 25, 2021

This is an error on Windows:

rellic/unittests/AST/ASTBuilder.cpp(64,21): error : call to constructor of 'llvm::APInt' is ambiguous
.../include/llvm/ADT/APInt.h(277,3): message : candidate constructor
.../include/llvm/ADT/APInt.h(304,3): message : candidate constructor

This is an error on Windows:

```
rellic/unittests/AST/ASTBuilder.cpp(64,21): error : call to constructor of 'llvm::APInt' is ambiguous
.../include/llvm/ADT/APInt.h(277,3): message : candidate constructor
.../include/llvm/ADT/APInt.h(304,3): message : candidate constructor
```
@pgoodman pgoodman merged commit 9bb2487 into master May 26, 2021
@pgoodman pgoodman deleted the fix-ambiguity-error-windows branch May 26, 2021 00:06
ekilmer added a commit that referenced this pull request May 29, 2021
* master:
  Fix links to cxx-common (#139)
  Fix constructor ambiguity error llvm::APInt in tests LLVM 11 (#138)
  Remove duplicate test run. CMake already runs both roundtrips.
  Refactor Clang AST building and fix typing in all representations (#121)
  Update anghabench-after-build.yml
  Disable 1M for now
  Update anghabench-after-build.yml
  Update anghabench-after-build.yml
  Run AnghaBench small runs in CI (#135)
  Add --install option to build script (#134)
  CI :Use v0.1.3 (latest) cxx-common prebuilt libraries (#133)
  Fixes a typo in IsSignExt() (#131)
  Adds handling of 128-bit integer literals (#130)
  Adds Z3 conversions from bool to bitvector sorts (#129)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants