WIP: Support LLVM 18 #483
Merged
WIP: Support LLVM 18 #483
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: xermicus <cyrill@parity.io>
Signed-off-by: xermicus <cyrill@parity.io>
Signed-off-by: Cyrill Leutwiler <bigcyrill@hotmail.com>
Signed-off-by: xermicus <cyrill@parity.io>
Signed-off-by: xermicus <cyrill@parity.io>
Signed-off-by: xermicus <cyrill@parity.io>
Signed-off-by: xermicus <cyrill@parity.io>
Signed-off-by: xermicus <cyrill@parity.io>
Signed-off-by: xermicus <cyrill@parity.io>
Signed-off-by: xermicus <cyrill@parity.io>
Signed-off-by: xermicus <cyrill@parity.io>
Signed-off-by: xermicus <cyrill@parity.io>
|
After a little poke, llvm-sys now has updated to support LLVM 18 |
Signed-off-by: xermicus <cyrill@parity.io>
|
Great, now I'm basically just waiting for an ubuntu build to use on CI |
|
I'm not sure what's going on with linux x86 binary LLVM releases, however the test suite passes locally for me. @TheDan64 the PR is ready to receive a review :) |
|
@xermicus wdym by the binarys? Tests seem to be working. Or was this an intermittent thing? |
|
Oh, there's no build check for LLVM 18, could you please add one @xermicus |
Closed
|
@TheDan64 yep I have it on my radar |
|
Brew just got LLVM 18, so maybe this has tricked through the ecosystem now? |
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Support LLVM 18.
Related Issue
#482
How This Has Been Tested
Option<Breaking Changes>
Checklist
Introduce feature flags
Wait out
llvm-sysrelease supporting LLVM 18Guard removed API methods and make test suite pass
Implement added functionality
Operand BundlesWe don't need this right now, can I leave this open for a future PR?I have read the Contributing Guide