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

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 22 10:41:58 PDT 2018


t.p.northover added a comment.

> Rather than a feature in `AArch.td`, I'd prefer a line in `AArch64Subtarget::initializeProperties()`.

I think I disagree. Putting it in AArch64Subtarget.cpp means that someone has to recompile clang to test this out on their CPU. I'd reserve initializeProperties for unfortunate situations where you can't easily set something on the command-line or in AArch64.td (i.e. non-bool options so far).


Repository:
  rL LLVM

https://reviews.llvm.org/D32564





More information about the llvm-commits mailing list