Add patch from upstream LLVM to fix ssa value names for long integer constants
Opened this issue · 1 comments
mhillenbrand commented
For constant ops from arithmetic constants, MLIR chooses SSA value names that reflect the constant. That failed for ints with > 64 bits as the function that derived that name used a shortcut that did not apply (and assert(), as here) for ints with more than 64 bits.
We have contributed a fix to upstream LLVM in llvm/llvm-project@1ef32e7 and need to add this patch into our build of LLVM 14.0.6. (the fix will be included in LLVM 15).
Surfaced by #43
vrpascuzzi commented
How is the CI passing? Locally, check-tests
fails with current main (c941dd9).