maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[WebAssebmly] Fully disable 'protected' visibility 

Emscripten doesn't use protected visibility either.

Differential Revision: https://reviews.llvm.org/D82346
This commit is contained in:
Sam Clegg 2020-06-22 20:30:40 -07:00
parent e7f7137cd7
commit 5804a8b122
1 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
clang/lib/Basic/Targets/WebAssembly.h
View File

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