[PATCH] D116655: [AArch64] Rename CPY to DUP. NFC

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 5 08:35:36 PST 2022


dmgreen added a comment.

In D116655#3222267 <https://reviews.llvm.org/D116655#3222267>, @tmatheson wrote:

> This renames the following instructions that were captured by the scheduler regex: CPYi8, CPYi16, CPYi32, CPYi64.
>
> The regex was also (unintentionally?) capturing the following SVE instructoins: CPY_ZPmI_B, CPY_ZPmI_D, CPY_ZPmI_H, CPY_ZPmI_S, CPY_ZPmR_B, CPY_ZPmR_D, CPY_ZPmR_H, CPY_ZPmR_S, CPY_ZPmV_B, CPY_ZPmV_D, CPY_ZPmV_H, CPY_ZPmV_S, CPY_ZPzI_B, CPY_ZPzI_D, CPY_ZPzI_H, CPY_ZPzI_S
>
> Do we need to do anything with them?

I though it was fine - the only cpu that has scheduling info and SVE instructions at the moment is the A64FX. That schedule has other lines for the CPY instructions, but looking again it looks like they are actually commented out.

I've uncommented them so they are the same as before.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116655/new/

https://reviews.llvm.org/D116655



More information about the llvm-commits mailing list