[PATCH] D75729: [RISCV] Select +0.0 immediate using fcvt.{d, s}.{x, w} fN, x0

Luís Marques via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 11 12:29:41 PDT 2020


luismarques added a comment.

> I'm happy to use those `fmv.{s,d}.x` instead.

I haven't yet had the opportunity to better investigate this. At first glance it seems fine to follow the GCC lead, and it could always be changed later, so if you want to update the patch for that I think I would be fine with approving it. But ideally we wouldn't just cargo cult this, and actually investigate what motivated that choice :-)
Thanks for sharing the table Roger.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75729





More information about the llvm-commits mailing list