[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