[PATCH] D119047: [OpaquePtr][LoopAccessAnalysis] Support opaque pointers

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 8 03:00:48 PST 2022


fhahn added inline comments.


================
Comment at: llvm/lib/Analysis/LoopAccessAnalysis.cpp:822
+      for (auto Access : Retries) {
+        for (auto &AccessTy : Accesses[Access]) {
+          if (!createCheckForAccess(RtCheck, Access, AccessTy, StridesMap,
----------------
I think we need at least test cases where there are actually more than on AccessTy. This seems not be be covered by the existing tests at the moment


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119047



More information about the llvm-commits mailing list