[WebAssebmly] Fully disable 'protected' visibility
authorSam Clegg <sbc@chromium.org>
Tue, 23 Jun 2020 03:30:40 +0000 (20:30 -0700)
committerSam Clegg <sbc@chromium.org>
Wed, 24 Jun 2020 00:50:05 +0000 (17:50 -0700)
Emscripten doesn't use protected visibility either.

Differential Revision: https://reviews.llvm.org/D82346

clang/lib/Basic/Targets/WebAssembly.h

index e09e21d..77a2fe9 100644 (file)
@@ -133,11 +133,7 @@ private:
 
   bool hasExtIntType() const override { return true; }
 
-  bool hasProtectedVisibility() const override {
-    // TODO: For now, continue to advertise "protected" support for
-    // Emscripten targets.
-    return getTriple().isOSEmscripten();
-  }
+  bool hasProtectedVisibility() const override { return false; }
 };
 
 class LLVM_LIBRARY_VISIBILITY WebAssembly32TargetInfo