ldionne added a comment.

As explained in more details here 
(https://reviews.llvm.org/D133249#inline-1289205), it would be nice to stop 
using the `DELETE.ME` idiom to kick off libc++ CI, unless you really think 
there's a good chance to cause failures in libc++ specifically.

TLDR of the link: the libc++ CI has 50+ jobs, exactly one of which will 
exercise your Clang changes. All the other jobs are using pre-compiled versions 
of Clang. So in reality, this `DELETE.ME` file trick only results in wasting 
libc++ CI resources, in addition to adding the libc++ review group on reviews 
that we probably don't need to look at. If you do suspect that the change might 
break libc++ specifically (such as changing diagnostics or whatever), then 
you're welcome to do so, although in the mid-term we'll try to simply run the 
libc++ tests as part of Clang's own CI setup.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134604

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to