[PATCH] D139601: [LoopVectorize] Clear cache of `LoopAccessInfoManager`

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 4 06:46:07 PST 2023


fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:10603
+  // Assure that the `LoopAccessManagerInfo` cache is invalidated.
+  LAIs->clear();
+
----------------
Saldivarcher wrote:
> @fhahn I wasn't sure where the best place to invalidate cache, if you think I should do it elsewhere I can do that. 
It *should* be sufficient to do it if `processLoop` at line 10632 made modifications I think


================
Comment at: llvm/test/Transforms/LoopVectorize/pr59319.ll:5
+target triple = "x86_64-unknown-linux-gnu"
+; REQUIRES: x86-registered-target
+
----------------
If it requires X86, please move it to the X86 subdirectory.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139601/new/

https://reviews.llvm.org/D139601



More information about the llvm-commits mailing list