[PATCH] D28922: [LoadCombine] Fix combining of loads which span an aliasing store.

Karl-Johan Karlsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 8 08:00:37 PST 2017


Ka-Ka added a comment.

Any news about this issue?



================
Comment at: test/Transforms/LoadCombine/load-combine-aa.ll:48
+; CHECK: load i16, i16*
+; CHECK: store i16
+; CHECK: ret i16
----------------
efriedma wrote:
> Check for the second load after the store?
The testcase run first the load-combine pass and then instcombine pass. The instcombine pass will remove the second load after the store. If you think it is important to check for the second load then the testcase need to be splitted from the other tests and the instcombine pass need to be turned off.



https://reviews.llvm.org/D28922





More information about the llvm-commits mailing list