[PATCH] D56888: [safestack] Replace statics with unnamed namespace

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 17 17:21:33 PST 2019


eugenis added inline comments.


================
Comment at: compiler-rt/lib/safestack/safestack.cc:267
 }
 
 extern "C" __attribute__((visibility("default")))
----------------
should the namespace end here?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56888





More information about the llvm-commits mailing list