[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