[PATCH] D28504: [WebAssembly] Only set one use at a time in FixFunctionBitcasts

Dan Gohman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 18:31:32 PST 2017


sunfish accepted this revision.
sunfish added a comment.
This revision is now accepted and ready to land.

Constants are meant to be immutable at some level in LLVM, so my idea with the RAUW was to avoid mutating them (RAUW has special logic for constants so that it doesn't actually mutate them). However, you're right that this breaks when there are multiple uses of the same function.


https://reviews.llvm.org/D28504





More information about the llvm-commits mailing list