[llvm-commits] [llvm] r167691 - /llvm/trunk/test/Transforms/InstCombine/memcmp-1.ll

Chris Lattner clattner at apple.com
Sun Nov 11 21:53:03 PST 2012


On Nov 11, 2012, at 4:49 PM, Meador Inge <meadori at codesourcery.com> wrote:

> 
> On Nov 11, 2012, at 4:07 PM, Eli Friedman wrote:
> 
>> This seems wrong: we prefer that transforms are consistent across
>> platforms where possible.  Please fix the transform so it doesn't vary
>> based on the return value of the system memcmp.
> 
> 
> Good point.  OK with the attached?

LGTM!  Thanks,

-Chris




More information about the llvm-commits mailing list