[PATCH] D34958: [AArch64] Add AArch64Subtarget::isFusion function.

Stefan Teleman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 11:04:05 PDT 2017


steleman added a comment.

In https://reviews.llvm.org/D34958#806717, @steleman wrote:

> In https://reviews.llvm.org/D34958#806341, @fhahn wrote:
>
> > Ping. @joelkevinjones @steleman are you happy for this to go in?
>
>
> I am running SPECint2K6 with and without your patch for comparison and
>  I will let you know as soon as I have all the numbers.


Your patch shows an improvement for 462.libquantum:

1. Without your patch: 315.392 = 65.695
2. With your patch: 301.580 = 68.704

The other benchmarks either didn't show much of a difference, or the difference
was well within the noise range.

So, I have no objections to this patch.


https://reviews.llvm.org/D34958





More information about the llvm-commits mailing list