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

Chad Rosier mcrosier at apple.com
Tue Apr 3 17:04:45 PDT 2012


Hi Rafael,
If you commit the instcombine patch and send me the specific revision, I'll watch our nightly testers for failures and regressions.

 Chad

On Apr 3, 2012, at 6:20 AM, Rafael EspĂ­ndola wrote:

>> It worked. I have committed the metadata bits. The updated patch for
>> instcombine is attached.
> 
> 
> In the attached patch I also changed the dag version of
> computeMaskedBits. I have run the test-sutite and I have attached the
> resulting json files, but unfortunately I am not able to compare them
> in lnt. A quick check with diff does show some improvements:
> 
>         {
>             "Data": [
> -                0.1333
> +                0.1256
>             ],
>             "Info": {},
>             "Name":
> "nts.MultiSource/Benchmarks/Trimaran/enc-md5/enc-md5.compile"
>         },
>         {
>             "Data": [
> -                1.5127
> +                1.5115
>             ],
>             "Info": {},
>             "Name": "nts.MultiSource/Benchmarks/Trimaran/enc-md5/enc-md5.exec"
>         },
> 
> Cheers,
> Rafael
> <t.patch><build-dag.json.bz2><build-master.json.bz2>





More information about the llvm-commits mailing list