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

Unified Diff: pnacl/driver/pnacl-ld.py

Issue 800553003: Update PNaCl to LLVM/Clang/Libcxx 223109 (nacl-change) (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: rebase again Created 5 years, 10 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
Index: pnacl/driver/pnacl-ld.py
diff --git a/pnacl/driver/pnacl-ld.py b/pnacl/driver/pnacl-ld.py
index 8f88171b01927dfc480c38be59850725381e4eb2..644985da4f50c86b36e64b1d5ef235b7c1dff8c5 100755
--- a/pnacl/driver/pnacl-ld.py
+++ b/pnacl/driver/pnacl-ld.py
@@ -37,7 +37,7 @@ EXTRA_ENV = {
'OPT_LEVEL': '', # Default opt is 0, but we need to know if it's explicitly
# requested or not, since we don't want to propagate
# the value to TRANSLATE_FLAGS if it wasn't explicitly set.
- 'OPT_LTO_FLAGS': '-std-link-opts -disable-internalize',
JF 2015/02/23 21:50:42 Why isn't -disable-internalize necessary anymore?
jvoung (off chromium) 2015/02/23 22:12:00 Well, the main thing is that -disable-internalize
+ 'OPT_LTO_FLAGS': '-std-link-opts',
'OPT_FLAGS': '${#OPT_LEVEL && !OPT_LEVEL == 0 ? ${OPT_LTO_FLAGS}} ' +
'-inline-threshold=${OPT_INLINE_THRESHOLD} ',

Powered by Google App Engine
This is Rietveld 408576698