[PATCH] D151396: [2/N][RISCV] Model vxrm in LLVM intrinsics and machine instructions for RVV fixed-point instructions

[Yueh-Ting (eop) Chen via Phabricator via llvm-commits]

eopXD marked an inline comment as done.
eopXD added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/rvv/vxrm.mir:28
     dead $x0 = PseudoVSETVLI killed renamable $x10, 197 /* e8, mf8, ta, ma */, implicit-def $vl, implicit-def $vtype
-    WriteVXRMImm 0, implicit-def $vxrm
-    renamable $v8 = PseudoVAADD_VV_MF8 killed renamable $v8, killed renamable $v9, $noreg, 3 /* e8 */, implicit $vxrm, implicit $vl, implicit $vtype
+    renamable $v8 = PseudoVAADD_VV_MF8 killed renamable $v8, killed renamable $v9, 0, $noreg, 3 /* e8 */, implicit $vxrm, implicit $vl, implicit $vtype
     PseudoRET implicit $v8
----------------
craig.topper wrote:
> eopXD wrote:
> > craig.topper wrote:
> > > Drop the `implicit $vxrm` here. It shouldn't exist until the pass runs.
> > > 
> > > The implicit $vl, implicit $vtype operands shouldn't exist until the InsertVSETVLI pass runs.
> > Removed `implicit $vxrm, implicit $vl, implicit $vtype`
> They still seem to be here?
My bad, I seem to mix up the commits. Removed it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151396