[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