[PATCH] D32564: AArch64: compress jump tables to minimum size needed to reach destinations

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 12 01:34:41 PDT 2018


rengolin added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64CompressJumpTables.cpp:145
+  if ((ST.getProcFamily() == AArch64Subtarget::ExynosM1 ||
+       ST.getProcFamily() == AArch64Subtarget::ExynosM3) &&
+      !MF->getFunction().optForMinSize())
----------------
Can we not add more Cpu name comparisons? Shouldn't be too hard to create a feature and associate it to these two cores. Later on, Samsung or any other would be able to fine tune to other cores. 


Repository:
  rL LLVM

https://reviews.llvm.org/D32564





More information about the llvm-commits mailing list