[PATCH] D80622: [LAA] Do not set CanDoRT to false for AS that do not need RT checks.
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Jun 6 06:54:49 PDT 2020
dmgreen added a comment.
This seems to make sense. Do you have a testcase? Maybe the one from D80570 <https://reviews.llvm.org/D80570> with some cleanup?
================
Comment at: llvm/lib/Analysis/LoopAccessAnalysis.cpp:763
+ if (!NeedsAliasSetRTCheck) {
+ assert(AS.size() <= 1 ||
+ all_of(AS,
----------------
I think you need brackets around the ||
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80622/new/
https://reviews.llvm.org/D80622
More information about the llvm-commits
mailing list