[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