[PATCH] D137324: [AMDGPU] Create new instructions in SIInstrInfo::moveToVALU

[Joe Nash via Phabricator via llvm-commits]

Joe_Nash added a comment.

This part of the commit message is worded confusingly for me.

"NFCI but it does have the minor side effect of not duplicating implicit
operands that are default for the VALU but were also present on the
original SALU."

I would probably change it to something like

NFCI but it does have the minor side effect of removing duplicate implicit
operands that were present on the
original SALU if they are default implicit operands for the VALU.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137324