[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