[PATCH] D65497: [RISCV] Avoid generating AssertZext for RV64 when lowering floating Libcall

Shiva Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 19:11:46 PDT 2019


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


================
Comment at: test/CodeGen/RISCV/rv32i-rv64i-float-double.ll:39
+; RV64IF-NEXT:    slli a1, s0, 32
+; RV64IF-NEXT:    srli a1, a1, 32
 ; RV64IF-NEXT:    call __divsf3
----------------
kito-cheng wrote:
> Why this two extension is not gone? I thought this patch should also improve divsf3 too?
Hi Kito,
I think you're right, it should be able to remove. I'll update the patch to catch the case, Thanks.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65497





More information about the llvm-commits mailing list