[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