[PATCH] D76620: [SYCL] Implement __builtin_unique_stable_name.

[Richard Smith - zygoloid via Phabricator via cfe-commits]

rsmith added a comment.

In D76620#2324858 <https://reviews.llvm.org/D76620#2324858>, @erichkeane wrote:

> The feature that this supports is a part of the SYCL 2020 Provisional Spec, I thought that was sufficient.  We'll look into an RFC to re-submit in the future.

Does that cover only functionality that can be implemented in terms of this builtin, or the builtin itself? If so, what is the functionality that needs to be supported? That information will be a good starting point for the RFC.

In D76620#2326499 <https://reviews.llvm.org/D76620#2326499>, @keryell wrote:

> Otherwise, just removing a feature used for almost 6 months will cause some kind of forking pain...

Well, there have at least not been any LLVM releases with this feature yet. (We're on the cusp of releasing LLVM 11, which would be the first release with this functionality.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76620