[PATCH] D116574: [RISCV] Materializing constants with 'rori'

[Baoshan Pang via Phabricator via llvm-commits]

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


================
Comment at: llvm/test/CodeGen/RISCV/imm.ll:11
+; RUN: llc -mtriple=riscv64 -riscv-disable-using-constant-pool-for-large-ints -mattr=+experimental-zbb \
+; RUN:   -verify-machineinstrs < %s | FileCheck %s -check-prefix=RV64IZBB
 
----------------
jrtc27 wrote:
> BaoshanPang wrote:
> > jrtc27 wrote:
> > > You've clearly not regenerated this file, just hacked it up by hand; use the script. Also probably better to put Zbb between Zba and Zbs?
> > I wasn't sure how to use the script.
> Did you regenerate the CHECK lines //before// reordering the RUN lines? Because the order doesn't match, and I suspect a subsequent regenerate will result in all the CHECK lines moving, making it hard to see actual future diffs.
Ok, I regenerated the file, now the order looks correct.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116574