[llvm] [NVPTX] Attempt to load params using symbol addition node directly (PR #119935)
Kevin McAfee via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 13 18:44:06 PST 2024
kalxr wrote:
> why don't we move the symbol to a register before using it in the add?
The main reason I prefer the current approach is that we can use the same code path for all cases. Inserting a MoveParam is undesirable in the general case, so we would only want to do it when DAG combines are disabled. Requiring a different path be used when an optimization is disabled because we generate incorrect code without the optimization seems to me something that is ideally avoided.
https://github.com/llvm/llvm-project/pull/119935
More information about the llvm-commits
mailing list