[PATCH] D30274: Remove CRC32 instructions from AArch64InstrInfo::hasShiftedReg

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 4 15:46:39 PST 2017


rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.

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!


Repository:
  rL LLVM

https://reviews.llvm.org/D30274





More information about the llvm-commits mailing list