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

Issue 873443004: PNaCl: Update LLVM and Subzero revision in pnacl/COMPONENT_REVISIONS (Closed)

Created:
5 years, 11 months ago by JF
Modified:
5 years, 11 months ago
Reviewers:
Jim Stichnoth
CC:
native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/native_client/src/native_client.git@master
Target Ref:
refs/heads/master
Project:
nacl
Visibility:
Public.

Description

PNaCl: Update LLVM and Subzero revision in pnacl/COMPONENT_REVISIONS This pulls in the following LLVM changes: 5d49470: (kschimpf@google.com) Allow temporary overrides of error stream in NaClBitcodeParser. 2c57bd8: (jfb@chromium.org) Fix more build warnings 97ac535: (dschuff@chromium.org) Remove unused LOCALMOD to disable libxml This pulls in the following Subzero changes: e3c02c2: (stichnot@chromium.org) Subzero: Disable asserts in the MINIMAL build. 586d4c2: (stichnot@chromium.org) Subzero: Improve performance by using llvm::SmallVector<>. a49e9d9: (stichnot@chromium.org) Subzero: Build with function/data sections and -gc-sections. 1c44d81: (stichnot@chromium.org) Subzero: Disable stats and timers under the MINIMAL build. f875d45: (kschimpf@google.com) Remove call to PNaClABIProps::isAllowedAlignment from subzero. 9bb188d: (kschimpf@google.com) Reduce one layer of decoding for binary operations in bitcode reading. 668a7a3: (stichnot@chromium.org) Subzero: Pull the node name out of the node structure. 1502e59: (stichnot@chromium.org) Subzero: Use llvm::ilist<> for PhiList and AssignList. 639c921: (stichnot@chromium.org) Subzero: Add memory usage to "-szstats" output. 9a04c07: (stichnot@chromium.org) Subzero: Don't store std::string objects inside Variable. 3281748: (kschimpf@google.com) Simplify LLVM's APInt and APFloat for use in Subzero. e5b73e6: (stichnot@chromium.org) Subzero: Clean up live range construction. bf17037: (kschimpf@google.com) Remove using LLVM tools to check correctness of cast operation. 4019f08: (kschimpf@google.com) Remove TypeConverter and Module from minimal subzero build. e6d2478: (stichnot@chromium.org) Subzero: Randomize register assignment. 31c9559: (stichnot@chromium.org) Subzero: Use CFG-local arena allocation for relevant containers. ae95320: (stichnot@chromium.org) Subzero: Convert NULL->nullptr. 29841e8: (stichnot@chromium.org) Subzero: Use range-based for loops with llvm::ilist<Inst> lists. c599e46: (stichnot@chromium.org) Subzero: Use the CfgLocalAllocator for more vector types. 5a122d7: (jvoung@chromium.org) Remove isPooled virtual method from IceOperand. 7e57136: (stichnot@chromium.org) Subzero: Clean up a few areas. 91a3e2c: (jvoung@chromium.org) Subzero: Write float/double constant pools directly to the ELF file. 2e7daef: (kschimpf@google.com) Add ability to test parsing of bitcode records in Subzero. 1d62cf0: (jvoung@chromium.org) Make fixups reference any constant (allow const float/double pool literals). 3018545: (stichnot@chromium.org) Subzero: Update the lowering documentation. e34d79d: (stichnot@chromium.org) Subzero: Use SmallVector<> instead of vector<> in a couple places. ec27073: (jvoung@chromium.org) Start writing out some relocation sections (text). 769be68: (stichnot@chromium.org) Subzero: Remove the IceV_RegManager enum value. af238b2: (kschimpf@google.com) Add instruction alignment tests to unit tests. e4a8f40: (stichnot@chromium.org) Subzero: Add locking to prepare for multithreaded translation. a086b91: (stichnot@chromium.org) Subzero: Remove the GlobalContext::GlobalDeclarations vector. 819d7b5: (kschimpf@google.com) Fix the PNaCl translator to lock the dump stream when printing errors. f2e93b6: (jfb@chromium.org) Fix build warnings 7d53825: (stichnot@chromium.org) Subzero: Make thread_local work under MacOS 10.6. fed97af: (jvoung@chromium.org) Make use of BSS more explicit in global initializers (vs a local .comm). a5fe17a: (stichnot@chromium.org) Subzero: Second attempt at fixing MacOS 10.6 build. BUG=none TEST= PNaCl toolchain trybots R=stichnot@chromium.org Committed: https://src.chromium.org/viewvc/native_client?view=rev&revision=2d65c2358780c5ace56b270743f04a5900c0ded7

Patch Set 1 #

Patch Set 2 : Pull in the subzero __thread fix. #

Patch Set 3 : Rebase. #

Patch Set 4 : Rebase properly. #

Patch Set 5 : Pull in the subzero OSX fix. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M pnacl/COMPONENT_REVISIONS View 1 2 3 4 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 15 (4 generated)
JF
5 years, 11 months ago (2015-01-22 22:42:24 UTC) #1
Jim Stichnoth
lgtm
5 years, 11 months ago (2015-01-22 22:43:20 UTC) #2
JF
C++11's TLS isn't supported by clang on OSX when the version is lower than 10.7, ...
5 years, 11 months ago (2015-01-22 23:44:07 UTC) #3
Jim Stichnoth
On 2015/01/22 23:44:07, JF wrote: > C++11's TLS isn't supported by clang on OSX when ...
5 years, 11 months ago (2015-01-23 18:26:32 UTC) #4
JF
On 2015/01/23 18:26:32, stichnot wrote: > On 2015/01/22 23:44:07, JF wrote: > > C++11's TLS ...
5 years, 11 months ago (2015-01-23 18:57:43 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/873443004/20001
5 years, 11 months ago (2015-01-23 18:57:47 UTC) #7
commit-bot: I haz the power
Failed to apply patch for pnacl/COMPONENT_REVISIONS: While running git apply --index -3 -p1; error: patch ...
5 years, 11 months ago (2015-01-23 18:57:50 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/873443004/60001
5 years, 11 months ago (2015-01-23 19:05:07 UTC) #11
commit-bot: I haz the power
Try jobs failed on following builders: nacl-win32_glibc_opt on tryserver.nacl (http://build.chromium.org/p/tryserver.nacl/builders/nacl-win32_glibc_opt/builds/9434)
5 years, 11 months ago (2015-01-23 19:08:59 UTC) #13
JF
I updated the CL again, this time to pull in stichnot's TLS fix for OSX ...
5 years, 11 months ago (2015-01-26 19:18:01 UTC) #14
JF
5 years, 11 months ago (2015-01-26 20:48:40 UTC) #15
Message was sent while issue was closed.
Committed patchset #5 (id:80001) manually as
2d65c2358780c5ace56b270743f04a5900c0ded7 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698