[PATCH] D130482: [LAA] Avoid non-determinism due to blocks order. PR56672

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 25 12:22:19 PDT 2022


fhahn added a comment.

I am not sure this patch completely fixes the underlying issue causing the mis-compile. I should know more by tomorrow and also be able to add a few cleaned up test cases.



================
Comment at: llvm/test/Transforms/LoopVectorize/X86/pr56672.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt -passes='loop(loop-rotate),loop-vectorize' -S -mcpu=skylake-avx512 -S %s | FileCheck %s
+; RUN: opt -passes='loop(loop-rotate),invalidate<loops>,loop-vectorize' -S -mcpu=skylake-avx512 -S %s | FileCheck %s
----------------
fhahn wrote:
> As this test a LAA issue, could you just use `print<loop-accesses>` instead?
>  use print<loop-accesses>

The pass name is actually `print-access-info`, which should probably be fixed.


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

https://reviews.llvm.org/D130482



More information about the llvm-commits mailing list