[PATCH] D138585: [SLPVectorizer] Do Not Move Loads/Stores Beyond Stacksave/Stackrestore Boundaries
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 28 08:38:22 PST 2022
reames added a comment.
Can you explain why memory dependence wasn't enough to prevent the problematic reordering? I'm a bit fuzzy on the details - it's been a while - but I though that stacksave and stackrestore were modeled as modifying all memory to prevent this reordering?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138585/new/
https://reviews.llvm.org/D138585
More information about the llvm-commits
mailing list