tahonermann accepted this revision.
tahonermann added a comment.

I'm not a regular Clang reviewer.  But for what it is worth, the changes look 
correct, clear, and appropriate from my lens (though I disagree slightly with 
some of the lint recommendations in the cases where the recommendation deviates 
from the surrounding code style).  It looks like the concerns Hubert raised 
have been addressed.


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

https://reviews.llvm.org/D82081

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

Reply via email to