[PATCH] D76620: [SYCL] Implement __builtin_unique_stable_name.

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Oct 11 20:58:08 PDT 2020


rjmccall added a comment.
Herald added a subscriber: yaxunl.

Richard and I just found out about this commit, and we've decided to revert it.  I apologize for the very late reversion, but the reality of Clang development is that it's very difficult for the code owners to watch literally every commit, and we rely on the community to follow policies and bring things to our attention when there's a question about the right thing to do.  This is a new feature that was submitted without properly following any of the guidelines for new features: it was neither approved by a relevant standards body, added for compatibility with an existing frontend, or taken through the RFC process.  If you had brought it to our attention as an RFC, we would've expressed serious concerns about the design.

Please start an RFC thread and CC Richard and me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76620



More information about the cfe-commits mailing list