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

Issue 10824040: NaCl: Update revision in DEPS, r9246 -> r9262 (Closed)

Created:
8 years, 4 months ago by Mark Seaborn
Modified:
8 years, 4 months ago
Reviewers:
bradn, Roland McGrath
CC:
chromium-reviews, native-client-reviews_googlegroups.com, Austin Benson, bsy, eaeltsin, jvoung - send to chromium..., Karl, Roland McGrath, Mark Seaborn, pasko-google - do not use, robertm
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r9246 -> r9262 This pulls in the following Native Client changes: r9247: (robertm) Roll archived pexe rev to compensate for a test change in rev 9235. r9248: (arbenson) Use AddBootstrap function where applicable r9249: (kschimpf) Update llvm revision: http://codereview.chromium.org/10808110 r9250: (robertm) Move pnacl toolchain size test from build script bot script. r9251: (mcgrathr) Get rid of __nacl_tp_alignment r9252: (robertm) Prune used binutils ld.bfd from pnacl toolchain. r9253: (mcgrathr) Fix TLS alignment calculation r9254: (jvoung) Emit a warning when bitcode file contains a known bug for ARM r9255: (jvoung) Remove debug listing code for pnacl/build.sh (related to r9256: (bsy) Add a developer hint log message for when RLIMIT_AS is the culprint in startup failure. r9257: (mseaborn) Remove the unused function NaClGetEbx() r9258: (mseaborn) Remove the unused file nacl_syscall_weak.S r9259: (mseaborn) Use the DEFINE_GLOBAL_HIDDEN_IDENTIFIER macro consistently in assembly code r9260: (eaeltsin) Debug stub: provide meaningful backtrace when stopped in syscall r9261: (mseaborn) Thread suspension: Create an internal API for getting and setting registers r9262: (pasko) validator_ragel: move the irt test from Makefile to scons BUG=none TEST=nacl_integration Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=148574

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Mark Seaborn
8 years, 4 months ago (2012-07-26 16:52:22 UTC) #1
Roland McGrath
lgtm
8 years, 4 months ago (2012-07-26 17:12:02 UTC) #2
bradn
LGTM On Thu, Jul 26, 2012 at 10:12 AM, <mcgrathr@chromium.org> wrote: > lgtm > > ...
8 years, 4 months ago (2012-07-26 18:35:14 UTC) #3
bradn
8 years, 4 months ago (2012-07-26 18:35:15 UTC) #4
LGTM


On Thu, Jul 26, 2012 at 10:12 AM, <mcgrathr@chromium.org> wrote:

> lgtm
>
>
>
>
http://codereview.chromium.**org/10824040/<http://codereview.chromium.org/108...
>

Powered by Google App Engine
This is Rietveld 408576698