[PATCH] D138585: [SLPVectorizer] Do Not Move Loads/Stores Beyond Stacksave/Stackrestore Boundaries

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 28 09:18:26 PST 2022


ABataev added a comment.

In D138585#3953869 <https://reviews.llvm.org/D138585#3953869>, @reames wrote:

> 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?

Did a quick search - looks like it does not work for static allocas, only for dynamic


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