[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