[PATCH] D83273: [X86] Remove the feature dependency handling in X86TargetInfo::setFeatureEnabledImpl to a table based lookup in X86TargetParser.cpp
Nick Desaulniers via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 4 16:10:09 PDT 2020
nickdesaulniers added a comment.
In D83273#2194832 <https://reviews.llvm.org/D83273#2194832>, @craig.topper wrote:
> It's a pretty nasty revert in our downstream tree where we have prototyped future ISAs. So I'd like a little time to take a look.
>
> @nickdesaulniers what cpu and fetaures are on your command lines. getImpliedDisabledFeatures should only be called if some feature is explicitly being disabled.
Likely, as the kernel is very explicit to disable the use of all FP extensions, see https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/x86/Makefile#n57. Maybe we could just set fewer flags, but there's still probably room for improvement here, too.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83273/new/
https://reviews.llvm.org/D83273
More information about the llvm-commits
mailing list