[PATCH] D113528: [RISCV] Fix non-sensical intrinsic names in rv64i-single-softfloat.ll. NFC

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 9 17:34:49 PST 2021


craig.topper added a comment.

In D113528#3120223 <https://reviews.llvm.org/D113528#3120223>, @jrtc27 wrote:

> I also don't understand why we're not getting tail calls here. Well, the intrinsic calls could be explained by using call rather than tail call in the IR, but I really don't get why the fadd etc instructions aren't converted to tail calls. Does LegalizeFloatTypes just fail to mark its libcalls as tail call candidates?

I think the TargetLowering::makeLibCall used by LegalizeFloatTypes doesn't set the tail call bit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113528



More information about the llvm-commits mailing list