[PATCH] D139507: [Intrinsic] Add get.rounding as alias to flt.rounds and rename related DAG nodes

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 08:51:43 PST 2022


RKSimon added a comment.

In D139507#3978449 <https://reviews.llvm.org/D139507#3978449>, @sepavloff wrote:

> Thank you for working on this!
>
> Is there any reason why we should keep the old intrinsic?

I'm not clear on the general policy, but for x86 we tend to provide bc of old intrinsics/sigtaures via AutoUpgrade.cpp, supposedly only for a few releases but tbh we remove them very rarely....


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139507



More information about the llvm-commits mailing list