[PATCH] D49563: [ARM] Add new target feature to fuse literal generation

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 27 10:06:19 PDT 2018


fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, this is in line with the Cortex-A57 and Cortex-A72 opt guides. We have a similar feature for AArch64 already.



================
Comment at: llvm/lib/Target/ARM/ARM.td:145
+// Fast execution of bottom and top halves of literal generation
+def FeatureFuseLiterals   : SubtargetFeature<"fuse-literals", "HasFuseLiterals", "true",
+                                             "CPU fuses literal generation operations">;
----------------
FeatureFuseLiterals should be added to the cortex-a57 and cortex-a72 features according to the optimization guides. Can you also add a run lines with those CPUs to the test?


Repository:
  rL LLVM

https://reviews.llvm.org/D49563





More information about the llvm-commits mailing list