njames93 added a comment.

In D61989#2314544 <https://reviews.llvm.org/D61989#2314544>, @collinjackson 
wrote:

> Are there any updates on this change? I see there's a comment that hasn't 
> been addressed yet. Is that the only roadblock stopping this from getting 
> merged?

Tbh clang-tidy already handles disabling warnings for header files, so some of 
the code here isn't needed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61989

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

Reply via email to