[PATCH] D126533: [LAA] Relax pointer dependency with runtime pointer checks

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 18 15:02:51 PDT 2022


fhahn added inline comments.


================
Comment at: llvm/test/Transforms/LoopDistribute/scev-inserted-runtime-check.ll:9
 
 define void @f(i32* noalias %a, i32* noalias %b, i32* noalias %c, i32* noalias %d, i32* noalias %e, i64 %N) {
 ; CHECK-LABEL: @f(
----------------
fhahn wrote:
> This test doesn't seem to be testing what it is supposed to any longer. I am not sure what changed and the new behavior isn't incorrect - we just don't version, but it would be good to know why and restore the original behavior for the test.
Did you manage to track down what changed here? Would it be possible to restore the original behavior of the test instead removing it?


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

https://reviews.llvm.org/D126533



More information about the llvm-commits mailing list