[PATCH] D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg
Azharuddin Mohammed via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 7 10:30:05 PST 2017
azharudd added a comment.
In https://reviews.llvm.org/D30274#692397, @rengolin wrote:
> Hi, right, so Tim has approved it on the mailing list, which didn't get transferred here.
>
> But he also said an additional interesting point why you have to use mcpu and not another mattr: this is a code path that only triggers with A53 because of its scheduler.
>
> LGTM, too. Can you just add the scheduler info (instead of just saying it hits with A53) on the commit line?
>
> thanks!
Updated the commit message to mention the A53 scheduler.
https://reviews.llvm.org/D30274
More information about the llvm-commits
mailing list