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

LLVM 15 #124

Merged
merged 1 commit into from
Nov 1, 2022
Merged

LLVM 15 #124

merged 1 commit into from
Nov 1, 2022

Conversation

langston-barrett
Copy link
Collaborator

Fixes #12. This may require changes to the Datalog code due to opaque pointers (#113).

@langston-barrett langston-barrett self-assigned this Nov 1, 2022
@langston-barrett
Copy link
Collaborator Author

Yep, some Datalog assertions are failing. I think there are a few options:

  • Try to fix the Datalog in this MR. This would be great, but could be a ton of effort.
  • Ask users to compile their code with -Xclang -no-opaque-pointers. I tried this and it didn't seem to fix the tests, so might be a no-go.
  • Wait until fixing dl: Adapt to opaque pointers #113 before supporting LLVM 15.
  • Keep LLVM 14 as the default, but support compiling against LLVM 15. I like this because the FactGenerator is independently useful to have building against more recent LLVM.

@langston-barrett langston-barrett force-pushed the lb/llvm15 branch 5 times, most recently from 8126604 to 8fbaab6 Compare November 1, 2022 16:41
@langston-barrett langston-barrett enabled auto-merge (rebase) November 1, 2022 17:03
@langston-barrett langston-barrett merged commit 4b3da07 into main Nov 1, 2022
@langston-barrett langston-barrett deleted the lb/llvm15 branch November 1, 2022 17:12
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.

Upgrade to the latest LLVM
1 participant