[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