scott.linder added a comment.

Richard Smith on cfe-dev pointed out some cases where this patch is incorrect, 
stemming from trying to calculate the linkage too early; the warning will 
either have to work without the use of `isExternallyVisible` or will have to be 
emitted later. This was reverted in `r359858` for now.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61097



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

Reply via email to