[PATCH] D75994: [GlobalISel] add additional lowering support for G_INSERT

[Daniel Sanders via Phabricator via llvm-commits]

dsanders added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-insert.mir:1298
     ; CHECK: [[OR:%[0-9]+]]:_(s32) = G_OR [[AND1]], [[AND]]
-    ; CHECK: [[BITCAST:%[0-9]+]]:_(s32) = G_BITCAST [[OR]](s32)
-    ; CHECK: $vgpr0 = COPY [[BITCAST]](s32)
----------------
gargaroff wrote:
> arsenm wrote:
> > Should this have been triggering a verifier error? Is there a missing check for source type == dest type?
> I guess so. I'd say that casting only makes sense when the types change. Might be worth to look into in a follow up.
I agree. G_BITCAST shouldn't be valid for the same type on operand and result


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75994