[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