This revision was automatically updated to reflect the committed changes. Closed by commit rG5804a8b1228b: [WebAssebmly] Fully disable 'protected' visibility (authored by sbc100).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82346/new/ https://reviews.llvm.org/D82346 Files: clang/lib/Basic/Targets/WebAssembly.h Index: clang/lib/Basic/Targets/WebAssembly.h =================================================================== --- clang/lib/Basic/Targets/WebAssembly.h +++ clang/lib/Basic/Targets/WebAssembly.h @@ -133,11 +133,7 @@ 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
Index: clang/lib/Basic/Targets/WebAssembly.h =================================================================== --- clang/lib/Basic/Targets/WebAssembly.h +++ clang/lib/Basic/Targets/WebAssembly.h @@ -133,11 +133,7 @@ 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
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits