[PATCH] D158256: [RISCV] Fix assertion failure when zcmp extension is enabled.

Anmol P. Paralkar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 25 09:07:15 PDT 2023


anmolparalkar-nxp added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/zcmp-prolog-epilog-crash.ll:15
+define hidden void @f0() local_unnamed_addr {
+; RV32I-LABEL: f0:                                     # @f0
+; RV32I:        .cfi_startproc
----------------
anmolparalkar-nxp wrote:
> Jim wrote:
> > Does it have any difference of codegen between RV32 and RV64? Can we use the same check-prefixes?
> No difference in codegen between RV32 and RV64; yes, agreed on the same check-prefixes. Thanks.
After incorporating @craig.topper's comments on the use of poison value and the write to 'ptr null' being code fragments that could potentially be optimized away in the future, I respun the code and now the code generated is different for RV32 and RV64; the checks themselves being auto-generated by utils/update_llc_test_checks.py


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158256



More information about the llvm-commits mailing list