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

Unified Diff: src/PNaClTranslator.cpp

Issue 1855683002: Subzero: Fix -timing-funcs and -timing-focus flags. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Rebase 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/IceGlobalContext.cpp ('k') | no next file » | 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 4aef2441742b42ce4129be856e45564e85109de0..0a8e52afc2e4436c82a8e507e0027c91ca410606 100644
--- a/src/PNaClTranslator.cpp
+++ b/src/PNaClTranslator.cpp
@@ -477,7 +477,14 @@ private:
Prefix);
} else {
Ice::GlobalContext *Ctx = Translator.getContext();
- if (Ice::BuildDefs::dump() || !Decl->isInternal()) {
+ // Synthesize a dummy name if any of the following is true:
+ // - DUMP is enabled
+ // - The symbol is external
+ // - The -timing-funcs flag is enabled
+ // - The -timing-focus flag is enabled
+ if (Ice::BuildDefs::dump() || !Decl->isInternal() ||
+ Ice::getFlags().getTimeEachFunction() ||
+ !Ice::getFlags().getTimingFocusOn().empty()) {
Decl->setName(Ctx, Translator.createUnnamedName(Prefix, NameIndex));
} else {
Decl->setName(Ctx);
« no previous file with comments | « src/IceGlobalContext.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698