[PATCH] D26811: [MemCpyOpt] Don't sink LoadInst below possible clobber.
bryant via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 22 17:23:34 PST 2016
bryant added a comment.
In https://reviews.llvm.org/D26811#603131, @efriedma wrote:
> LGTM with the testcase fixed... but please don't commit changes to this code unless you're intending to write a followup which actually makes it useful. (If you're going to leave it in its current state, we might as well just delete it.)
Not sure that I understand. Are you suggesting that we leave the bug unfixed?
Repository:
rL LLVM
https://reviews.llvm.org/D26811
More information about the llvm-commits
mailing list