[PATCH] D139507: [Intrinsic] Add get.rounding as alias to flt.rounds and rename related DAG nodes
Nikita Popov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Dec 8 00:25:39 PST 2022
nikic requested changes to this revision.
nikic added a comment.
This revision now requires changes to proceed.
In D139507#3980555 <https://reviews.llvm.org/D139507#3980555>, @qiucf wrote:
> 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?
>
> In case any user outside of clang references it (although I believe no), we can deprecate it and remove after a few releases.
This is not how intrinsic changes work. You need to add AutoUpgrade support.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139507/new/
https://reviews.llvm.org/D139507
More information about the cfe-commits
mailing list