Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(21)

Unified Diff: src/PNaClTranslator.cpp

Issue 1766233002: Subzero: Fix symbol name mangling. Make flags global. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Code review changes Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/IceTargetLoweringX86BaseImpl.h ('k') | unittest/BitcodeMunge.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/PNaClTranslator.cpp
diff --git a/src/PNaClTranslator.cpp b/src/PNaClTranslator.cpp
index 1fb84c16b76e32f44592ed8670210b7e12031439..595b2340eb515b1ed3878e3596ebecc26e2bded0 100644
--- a/src/PNaClTranslator.cpp
+++ b/src/PNaClTranslator.cpp
@@ -508,16 +508,15 @@ private:
}
}
- // Builds a constant symbol named Name, suppressing name mangling if
- // SuppressMangling. IsExternal is true iff the symbol is external.
+ // Builds a constant symbol named Name. IsExternal is true iff the symbol is
+ // external.
Ice::Constant *getConstantSym(const Ice::IceString &Name,
- bool SuppressMangling, bool IsExternal) const {
+ bool IsExternal) const {
if (IsExternal) {
return getTranslator().getContext()->getConstantExternSym(Name);
} else {
const Ice::RelocOffsetT Offset = 0;
- return getTranslator().getContext()->getConstantSym(Offset, Name,
- SuppressMangling);
+ return getTranslator().getContext()->getConstantSym(Offset, Name);
}
}
@@ -538,8 +537,7 @@ private:
for (const Ice::FunctionDeclaration *Func : FunctionDeclarations) {
if (!Func->verifyLinkageCorrect(Ctx))
reportLinkageError("Function", *Func);
- Ice::Constant *C = getConstantSym(
- Func->getName(), Func->getSuppressMangling(), Func->isProto());
+ Ice::Constant *C = getConstantSym(Func->getName(), Func->isProto());
ValueIDConstants.push_back(C);
}
}
@@ -551,8 +549,7 @@ private:
if (!Decl->verifyLinkageCorrect(Ctx))
reportLinkageError("Global", *Decl);
Ice::Constant *C =
- getConstantSym(Decl->getName(), Decl->getSuppressMangling(),
- !Decl->hasInitializer());
+ getConstantSym(Decl->getName(), !Decl->hasInitializer());
ValueIDConstants.push_back(C);
}
}
« no previous file with comments | « src/IceTargetLoweringX86BaseImpl.h ('k') | unittest/BitcodeMunge.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698