[PATCH] D23446: [X86] Enable setcc to srl(ctlz) transformation on btver2 architectures.
pierre gousseau via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 5 10:05:58 PDT 2016
pgousseau added inline comments.
> RKSimon wrote in X86ISelLowering.cpp:29016
> Should we return early if !Subtarget.isCtlzFast()?
Yes that makes sense.
> RKSimon wrote in X86ISelLowering.cpp:29022
> Are we letting vector types through here?
Yes it seems that way, will have to restrict to 32-bit and 64-bit integers, thanks for spotting this.
> RKSimon wrote in X86ISelLowering.cpp:31797
> Comment this. Should this be part of a separate patch with its own tests? Its fine if its only exposed by this patch to leave it here, but it should have a comment either way.
Yes would make sense to have a separate patch, will try to find an associated test case.
> RKSimon wrote in lzcnt-zext-cmp.ll:7
> Test single input version to make sure it isn't being used.
Makes sense.
https://reviews.llvm.org/D23446
More information about the llvm-commits
mailing list