[PATCH] D17298: [WebAssemly] Don't move calls or stores past intervening loads
Dan Gohman via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 16 12:41:17 PST 2016
sunfish accepted this revision.
sunfish added a comment.
This revision is now accepted and ready to land.
Good catch. The isSafeToMove API is pretty confusing; this is what comes of trying to be clever with it. I think we'll eventually want to revisit this code and do something other than isSafeToMove, but this change looks ok for now.
http://reviews.llvm.org/D17298
More information about the llvm-commits
mailing list