Skip to content

Conversation

@ingomueller-net
Copy link
Collaborator

@ingomueller-net ingomueller-net commented Jan 30, 2025

This PR updates the LLVM git submodule to the latest version as of January 31, 2025, namely llvm/llvm-project@956cfa69b15 and adapts the code base to the new version. The concrete motivation for the update is llvm/llvm-project#124770, which fixes the TypedStrAttr definition that #73 uses.

This PR requires to update the git modules in existing git clones and to install new pip dependencies:

git submodule update --recursive
pip install -r requirements.txt

@ingomueller-net ingomueller-net changed the title feat: update LLVM to latest version as of Jan. 31, 2025 (956cfa69b15) feat: update LLVM to latest version as of Jan. 31, 2025 (956cfa69b15) Jan 30, 2025
This PR updates the LLVM git submodule to the latest version as of
January 31, 2025, namely llvm/llvm-project@956cfa69b15.

Signed-off-by: Ingo Müller <[email protected]>
The way `FloatType`s are built was changed upstream: instead of
`FloatType::getF32`, we now use `Float32Type::get`. This commit adapts
the usages of the previous pattern. While touching that code, the commit
also refactors both `exportFloatType` and `exportIntegerType` to use a
`switch` statement on the type width instead of constructing all
possible MLIR types and comparing linearly against them, which is less
efficient.

Signed-off-by: Ingo Müller <[email protected]>
@dshaaban01
Copy link
Collaborator

@dshaaban01

@ingomueller-net ingomueller-net merged commit 14fb4c1 into substrait-io:main Feb 7, 2025
7 checks passed
@ingomueller-net ingomueller-net deleted the update-llvm branch February 7, 2025 13:39
@ingomueller-net
Copy link
Collaborator Author

This PR and #79 had a hidden dependency that I didn't catch before merging, so now CI is broken. #85 fixes the breakage. Until that is merged, we need to rebase PRs onto that to make them pass CI.

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.

3 participants