[PATCH] D69891: [VP,Integer,#1] Vector-predicated integer intrinsics

[Andy Kaylor via Phabricator via llvm-commits]

andrew.w.kaylor added a subscriber: craig.topper.
andrew.w.kaylor added a comment.

In D69891#1871485 <https://reviews.llvm.org/D69891#1871485>, @simoll wrote:

> 5. ** Rename to `llvm.experimental.vp.*` ** - Inserting `experimental` is the preferred way to introduce new intrinsics until they are stable (for technical reasons brought up by @chandlerc -  https://reviews.llvm.org/D69891#1852795 ).


I feel like I should mention that I don't know how Chandler feels about the use of "experimental" for these intrinsics. I wasn't trying to claim his approval of my suggestion, merely explaining that I thought the reasoning that led to the current naming of the constrained FP intrinsics probably applied here as well. After I made that suggestion @craig.topper pointed out to me that the "experimental" qualifier has a tendency to never go away. See, for example, vector add/reduce intriniscs. All this is to say that my suggestion is just a suggestion and I could be convinced to drop it if that is the consensus.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69891