[llvm] [MachinePipeliner] Fix MachineMemOperand to be cleared when update fails (PR #80841)
Yuta Mukai via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 7 04:41:59 PST 2024
ytmukai wrote:
@davemgreen Thanks for your comments, I've reconsidered the load/store code should be modified.
It guarantees that the address of ldp/stp is aligned to twice the element size by MachineMemOperand if it needs to be. However, it assumes that the alignment is guaranteed if MachineMemOperand does not exist or has an invalid value, which appears to be a problem.
I will propose a modification that would determine it more conservatively.
https://github.com/llvm/llvm-project/blob/ca1da36aec963c5504ae7ae2e834b37856c476db/llvm/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp#L2347-L2362
https://github.com/llvm/llvm-project/pull/80841
More information about the llvm-commits
mailing list