[PATCH] D18812: [LLD] Fix Clang-tidy readability-static-definition-in-anonymous-namespace warnings; other minor fixes.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 5 19:34:47 PDT 2016


ruiu added a comment.

I think as a policy we should use `static` for functions instead of anonymous namespaces in LLVM. Can you narrow the scopes of anonymous namespaces instead of removing `static`?


Repository:
  rL LLVM

http://reviews.llvm.org/D18812





More information about the llvm-commits mailing list