dblaikie added a comment.

@aaron.ballman wouldn't mind your take on this to see if this seems 
sufficiently robust, tested, etc. (should I move the isExternallyVisible check 
even further down? So its side effects are even less impactful (maybe there are 
other warnings that care about this sort of thing?) )


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121328

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

Reply via email to