[PATCH] D82987: [BasicAA] Fix recursive phi MustAlias calculations

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 1 12:27:03 PDT 2020


efriedma added a comment.

Commit message uses "NoAlias" where it means to use "MayAlias"?



================
Comment at: llvm/test/Analysis/BasicAA/recphi.ll:8
 ; CHECK:         NoAlias:      float* %next, float* %src1
-; CHECK:         NoAlias:      float* %next, float* %src2
+; CHECK:         MayAlias:     float* %next, float* %src2
 ; CHECK:         NoAlias:      float* %next, float* %phi
----------------
`NoAlias:      float* %next, float* %src2` is actually correct; not sure how hard it would be to prove, though.


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

https://reviews.llvm.org/D82987





More information about the llvm-commits mailing list