[PATCH] D34142: [ARM] Add macro fusion for AES instructions.

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 14 09:27:05 PDT 2017


evandro added inline comments.


================
Comment at: lib/Target/ARM/ARMTargetMachine.cpp:399
+    const ARMSubtarget &ST = C->MF->getSubtarget<ARMSubtarget>();
+    if (ST.hasFuseAES())
+      DAG->addMutation(createARMMacroFusionDAGMutation());
----------------
Granted that `fuse-aes` is currently the only fusion, but I wonder if it´d make sense to create a more encompassing `hasFusion()` method that would combine all and any instruction fusion.


https://reviews.llvm.org/D34142





More information about the llvm-commits mailing list