[llvm-commits] [pr1225] Fix using instcombine instead of instsimplify

Chad Rosier mcrosier at apple.com
Wed Apr 4 09:16:27 PDT 2012


On Apr 4, 2012, at 6:15 AM, Rafael EspĂ­ndola wrote:

> 2012/4/3 Chad Rosier <mcrosier at apple.com>:
>> Hi Rafael,
>> If you commit the instcombine patch and send me the specific revision, I'll watch our nightly testers for failures and regressions.
> 
> Thanks!
> 
> I did a run with --multisample 20, but the program that was showing
> the largest difference still had the same md5 before and after the
> patch :-(
> 
> I compared the assembly of produced .o files before and after (only 34
> of them changed). The differences were what I was expecting: some
> 'and' instructions got removed and in some of those cases codegen got
> a bit confused.
> 
> The new revision is 154011.

Thanks, Rafael.  The commit didn't make last night's round of testing, so I'll let you know how it goes tomorrow morning.

 Chad

> 
>>  Chad
>> 
> 
> Cheers,
> Rafael





More information about the llvm-commits mailing list