[PATCH] D118102: [LoopInterchange] Detect output dependency of a store instruction with itself

[Bardia Mahjour via Phabricator via llvm-commits]

bmahjour requested changes to this revision.
bmahjour added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/test/Transforms/LoopInterchange/lcssa-preheader.ll:5
 target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
-
-; void foo(int n, int m) {
----------------
This is a valid test case to interchange, and looks like after this patch, it won't be interchanged anymore. I believe @congzhe is looking into seeing if we can treat scalar output dependencies as interchange-preventing so that we can still get scenarios like this one.


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

https://reviews.llvm.org/D118102