[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