[all-commits] [llvm/llvm-project] 343e20: [ARM] Replace TransferImpOps with copyImplicitOps

[john-brawn-arm via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 343e204a52845dcd7bb7e7b8213513bcc33939c8
      https://github.com/llvm/llvm-project/commit/343e204a52845dcd7bb7e7b8213513bcc33939c8
  Author: John Brawn <john.brawn at arm.com>
  Date:   2023-07-18 (Tue, 18 Jul 2023)

  Changed paths:
    M llvm/lib/Target/ARM/ARMExpandPseudoInsts.cpp
    M llvm/test/CodeGen/ARM/expand-pseudos.mir
    A llvm/test/CodeGen/Thumb2/expand-pseudos.mir

  Log Message:
  -----------
  [ARM] Replace TransferImpOps with copyImplicitOps

In most places where TransferImpOps is currently used we just have one
machine instruction, so it's doing the same thing as copyImplicitOps
anyway. In those cases where we have more than one machine
instruction the destination is written to in each instruction so any
implicit defs should appear on all of them (and we shouldn't see any
implicit refs as these pseudo-instruction don't have any register
inputs), meaning the current use of TransferImpOps is incorrect and
we should be using copyImplicitOps on all of the generated
instructions.

Differential Revision: https://reviews.llvm.org/D155301