erichkeane added a comment.

In D122822#3421089 <https://reviews.llvm.org/D122822#3421089>, @yihanaa wrote:

> I also agree that this is a useful debugging builtin, and no program logic 
> should be forced to depend on the output of this builtin. It's just used to 
> print out some useful debugging information

FWIW, @rsmith is the clang-codeowner, so he's the one to convince (and 
@rjmccall is also a very senior reviewer).  So I wouldn't be able to approve 
this patch without their consent/agreement.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122822

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

Reply via email to