[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