openqasm/qe-compiler

Add patch from upstream LLVM to fix ssa value names for long integer constants

Opened this issue · 1 comments

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

How is the CI passing? Locally, check-tests fails with current main (c941dd9).