carlosgalvezp added a comment.

Kind ping to reviewers, I mostly want to know if you agree with the general 
direction so I can go ahead and implement the rest of the patch. It's a big 
change (replace all check registrations with the macro) so I'd like to avoid 
extra work if you are strongly against it.

@aaron.ballman I understand you are super busy and don't want to take any more 
time than needed from you. I'd just love to hear your thoughts about the two 
missing points that we discussed in the RFC. Nobody else seems to have raised 
similar concerns about it.

Really appreciate your time!


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

https://reviews.llvm.org/D114317

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

Reply via email to