[llvm] r255896 - [AArch64] Enable PostRAScheduler for AArch64 generic build

[Rafael Espíndola via llvm-commits]

This broke multiple tests, so I reverted it.

Cheers,
Rafael


On 17 December 2015 at 09:51, MinSeong Kim via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: minseongkim
> Date: Thu Dec 17 08:51:22 2015
> New Revision: 255896
>
> URL: http://llvm.org/viewvc/llvm-project?rev=255896&view=rev
> Log:
> [AArch64] Enable PostRAScheduler for AArch64 generic build
>
> This patch enables PostRAScheduler specifically for AArch64 generic build,
> which is beneficial from the performance perspective.
> Speedups up to 2 to 7% for some benchmarks on A57 and A53 are observed.
> Also benchmarks from LLVM test-suite did not regress.
>
> Differential Revision: http://reviews.llvm.org/D15557
>
> Modified:
>     llvm/trunk/lib/Target/AArch64/AArch64Subtarget.h
>
> Modified: llvm/trunk/lib/Target/AArch64/AArch64Subtarget.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64Subtarget.h?rev=255896&r1=255895&r2=255896&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/AArch64/AArch64Subtarget.h (original)
> +++ llvm/trunk/lib/Target/AArch64/AArch64Subtarget.h Thu Dec 17 08:51:22 2015
> @@ -102,7 +102,7 @@ public:
>    const Triple &getTargetTriple() const { return TargetTriple; }
>    bool enableMachineScheduler() const override { return true; }
>    bool enablePostRAScheduler() const override {
> -    return isCortexA53() || isCortexA57();
> +    return isGeneric() || isCortexA53() || isCortexA57();
>    }
>
>    bool hasV8_1aOps() const { return HasV8_1aOps; }
> @@ -139,6 +139,7 @@ public:
>    bool isTargetELF() const { return TargetTriple.isOSBinFormatELF(); }
>    bool isTargetMachO() const { return TargetTriple.isOSBinFormatMachO(); }
>
> +  bool isGeneric() const { return CPUString == "generic"; }
>    bool isCyclone() const { return CPUString == "cyclone"; }
>    bool isCortexA57() const { return CPUString == "cortex-a57"; }
>    bool isCortexA53() const { return CPUString == "cortex-a53"; }
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits