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

Bump to llvm/llvm-project@3f1486f #19683

Merged
merged 11 commits into from
Jan 17, 2025
Merged

Conversation

MaheshRavishankar
Copy link
Contributor

@MaheshRavishankar MaheshRavishankar commented Jan 13, 2025

New changes

  • We're starting to pick up TOSA dialect v1.0 changes llvm/llvm-project@360a03c and llvm/llvm-project@c1d01b2 . These are breaking changes so existing TOSA .mlir[bc] files may not work with new compiler versions. These changes will also be propagating to framework exporters/importers like TensorFlow over time, so more breaks are expected. The iree-import-tflite tool now reports a warning if a known-incompatible version of tensorflow is installed.

Reverts

RISC-V backend

Python related changes

NVPTX changes

Updates to Torch-MLIR

Signed-off-by: MaheshRavishankar <mahesh.ravishankar@gmail.com>
Signed-off-by: MaheshRavishankar <mahesh.ravishankar@gmail.com>
Signed-off-by: MaheshRavishankar <mahesh.ravishankar@gmail.com>
Signed-off-by: MaheshRavishankar <mahesh.ravishankar@gmail.com>
@MaheshRavishankar MaheshRavishankar marked this pull request as ready for review January 13, 2025 23:30
Signed-off-by: MaheshRavishankar <mahesh.ravishankar@gmail.com>
@MaheshRavishankar MaheshRavishankar force-pushed the shared/integrate/20250112 branch from 34c932f to 0d57e70 Compare January 14, 2025 01:05
…ix Bazel build

Signed-off-by: Krzysztof Drewniak <Krzysztof.Drewniak@amd.com>
@krzysz00 krzysz00 force-pushed the shared/integrate/20250112 branch from b55de5b to d41b0d1 Compare January 16, 2025 20:55
@ScottTodd ScottTodd mentioned this pull request Jan 16, 2025
6 tasks
LLVM PR #121834 tries to get CPU flags from the command line options,
which somehow works OK in most builds even if the flags aren't
registered, but fails in ASan builds, so we revert here for now.

Signed-off-by: Krzysztof Drewniak <Krzysztof.Drewniak@amd.com>
Signed-off-by: Scott Todd <scott.todd0@gmail.com>
@ScottTodd ScottTodd force-pushed the shared/integrate/20250112 branch from b5c7236 to a745991 Compare January 16, 2025 22:12
@krzysz00 krzysz00 force-pushed the shared/integrate/20250112 branch from ea624f8 to 2ab8e4b Compare January 17, 2025 18:58
Signed-off-by: Krzysztof Drewniak <Krzysztof.Drewniak@amd.com>
Signed-off-by: Krzysztof Drewniak <Krzysztof.Drewniak@amd.com>
@krzysz00 krzysz00 force-pushed the shared/integrate/20250112 branch from bb4e3d4 to 208ff2c Compare January 17, 2025 20:10
Copy link
Contributor

@qedawkins qedawkins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codegen side LGTM. I have no idea what's going on in TOSA/TF though :P

@MaheshRavishankar MaheshRavishankar merged commit 6052a1d into main Jan 17, 2025
40 checks passed
@MaheshRavishankar MaheshRavishankar deleted the shared/integrate/20250112 branch January 17, 2025 21:45
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.

4 participants