-
Notifications
You must be signed in to change notification settings - Fork 294
Fix LLVM aarch64 relocation overflow #1444
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
Conversation
Diffs originated from llvm/llvm-project#71968 and were modified to target the specific version of LLVM in use by CUDA Quantum (16.0.6).
|
Note: need to revert when LLVM is rebased. |
That PR hasn't been merged in upstream LLVM yet, so we may need to rebase it (assuming this works in the first place). |
|
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
|
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Hopefully this gets merged upstream fairly soon so this will just be fixed out of the box. |
|
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
Fixes #1421
#1421 appears similar to llvm/llvm-project#71963, which is possibly fixed by llvm/llvm-project#71968.
The diffs in this PR originated from llvm/llvm-project#71968 and were modified to target the specific version of LLVM in use by CUDA Quantum (16.0.6). We also changed the default value of ReserveAlloc from false to true.
I did some UCCSD timings with and without this change on x86 / H100's, nvidia-fp64 simulator. The table below is an average of 10 runs. All times are in seconds.
The 12 qubit case is the only one that looks noticeably worse, and that one appeared to have large run-to-run variance (32ms - 138ms).