[PATCH] D90973: [RISCV] Remove RV32 HwMode. Use DefaultMode for RV32

[Jessica Clarke via Phabricator via llvm-commits]

jrtc27 added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCV.td:182-183
 
+// RV32 uses DefaultMode.
 def RV64           : HwMode<"+64bit">;
 
----------------
craig.topper wrote:
> jrtc27 wrote:
> > craig.topper wrote:
> > > lenary wrote:
> > > > lenary wrote:
> > > > > Is there a way of doing an alias here, so we don't have to document that defaultmode is rv32 everywhere?
> > > > > 
> > > > > Something like the following (I guess it has to be a let, not a def, but I could be wrong).
> > > > > ```
> > > > > let RV32 = DefaultMode;`
> > > > > ```
> > > > Sorry, it can't be a let, but you could do the following, I think
> > > > ```
> > > > defvar RV32 = DefaultMode;
> > > > ```
> > > > 
> > > > And then in an example pulled from below:
> > > > ```
> > > > def XLenVT : ValueTypeByHwMode<[RV64, RV32],
> > > >                                [i64,  i32]>;
> > > > ```
> > > That works, so I'll switch to that.
> > Hm, does this work the other way round too (and if not, what happens)? Having RV64 before RV32 is not the usual order.
> It does! Switched to that order in 9211da4215b6d48c8b9186b78274946789c559e9
Thanks! I always expect the worst when HwMode is involved...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90973