[PATCH] D117806: [OpenMP] Change default visibility to protected for device declarations

Joseph Huber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 20 12:20:45 PST 2022


jhuber6 added inline comments.


================
Comment at: clang/test/OpenMP/declare_target_codegen.cpp:293
-};
-int SS::SSS = 1;
 #endif
----------------
jdoerfert wrote:
> What happened here?
That was a special case I added that was only necessary when we were trying to remove things from being hidden. Now that hidden is not the default it's not necessary anymore.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117806



More information about the cfe-commits mailing list