[llvm-commits] [llvm] r114781 - /llvm/trunk/lib/Transforms/Scalar/MemCpyOptimizer.cpp

[Eric Christopher]

On Sep 24, 2010, at 9:20 PM, Jim Grosbach wrote:

> Eesh. That's a nasty one. Nice fix.
> 
> re: the TODO, I suspect the answer is "maybe." Not sure off the top of my head how to accurately model the change in cost, though.

Yeah, we'd have to model whether or not we think it'll be combined with another memcpy, blah blah blah, etc.

Probably worth just removing the TODO and hoping for the best :)

-eric