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

Javed Absar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 4 01:10:34 PDT 2017


javed.absar added inline comments.


================
Comment at: lib/Target/AArch64/AArch64TargetMachine.cpp:290
     const AArch64Subtarget &ST = C->MF->getSubtarget<AArch64Subtarget>();
-    if (ST.hasFuseAES() || ST.hasFuseLiterals()) {
+    if (ST.hasFusion()) {
       // Run the Macro Fusion after RA again since literals are expanded from
----------------
This doesn't look like NFCI - as previously this would be triggered on hasFuseAES and hasFuseLiteral only, but now it will include Bcc and  cbz fusion as well.


https://reviews.llvm.org/D34958





More information about the llvm-commits mailing list