[PATCH] D17289: [X86] Fix False Data Dependency in popcnt
Simon Pilgrim via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 17 06:35:05 PST 2016
RKSimon added subscribers: RKSimon, spatel, andreadb.
RKSimon added a comment.
Your current implementation would affect AMD Jaguar / Bulldozer families as well despite them not suffering from the dependency bug.
It looks like this might need to be hidden behind a feature bit (e.g. FeaturePOPCNTFalseDependency) instead of trying to decode the cpu from the target bits.
I should also add that there is a discussion on PR26183 about whether these types of fixes should all be put under the MachineCombiner pass.
Repository:
rL LLVM
http://reviews.llvm.org/D17289
More information about the llvm-commits
mailing list