[PATCH] D58311: [MemorySSA & LoopPassManager] Enable MemorySSA as loop dependency. Update tests.

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 6 21:28:53 PDT 2019


asbirlea marked 3 inline comments as done.
asbirlea added inline comments.


================
Comment at: test/Analysis/BasicAA/store-promote.ll:32
 ; CHECK:      Out:
-; CHECK:        store i32 %Atmp, i32* @B
+; CHECK:        load i32, i32* @A
 }
----------------
chandlerc wrote:
> Is this actually an additional load? Or was it always there before but the test just didn't have a `CHECK` line for it?
It's not an additional load, it was always there and there was no `CHECK` line.
The test difference is that the store is hoisted instead of sunk.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58311





More information about the llvm-commits mailing list