[PATCH] D110812: [test] Pre-commit test case for PR51981. NFC

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope added inline comments.


================
Comment at: llvm/test/Transforms/LoopRotate/pr51981-scev-problem.ll:3
+; RUN: opt < %s -passes='print<scalar-evolution>,loop(loop-rotate),print<scalar-evolution>' -disable-output 2>&1 | FileCheck -check-prefixes CHECK-SCEV,CHECK-SCEV-NOK %s
+; FIXME (crashes): opt < %s -passes='loop(canon-freeze),loop(loop-rotate),function(bounds-checking)' -disable-output
+
----------------
fhahn wrote:
> Just do double check, running `loop(canon-freeze)` is required for the crash? Is that only so the SCEV cache is populated?
Yes, that is my understanding. I tried to populate it using `print<scalar-evolution>` instead, or `require<scalar-evolution>`, but then I do not see the crash for some reason. 
Btw, I noticed right now that I can replace `bounds-checking` by `print<scalar-evolution>` and then I still get the crash (but in the printing pass instead).

Not quite sure why the symptom is a bit different depending on how I populate the cache. Afaict canon-freeze isn't doing any modifications to the IR (and neither should the printing).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110812