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

Issue 10837160: NaCl: Update revision in DEPS, r9391 -> r9403 (Closed)

Created:
8 years, 4 months ago by Mark Seaborn
Modified:
8 years, 4 months ago
Reviewers:
bradn
CC:
chromium-reviews, native-client-reviews_googlegroups.com, Austin Benson, Brad Chen, khimg, Mark Seaborn, petarj, Petr Hosek, robertm, Vlad Shcherbina
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r9391 -> r9403 This pulls in the following Native Client changes: r9392: (khim) Speedup the check-decoder test r9393: (shcherbina) Validator_ragel: fix validator_test segfault on no input files r9394: (bradchen) Restrict underdefined bit fields for lfence, mfence and sfence. r9395: (mseaborn) Debug stub: Make HLT produce the same signal on Windows and Linux r9396: (arbenson) Allocate zero-based memory if prereserved memory found on x86-64 r9397: (robertm) Make a function at a time translation the default for pnacl. r9398: (bradchen) Minor clean to enuminst; fixes a base register analysis error. r9399: (robertm) Increase test coverage on pnacl bot. r9400: (bradchen) Fix ragel checking of unsandboxed uses of RSI/RDI and RBP/RSP. r9401: (phosek) SConstruct fix for OS X 10.8 and newer which deprecated and removed older OS X SDKs. r9402: (petarj) [MIPS] Toolchain scripts for MIPS architecture. r9403: (arbenson) Pull out NaClVmmapAdd calls in NaClMprotectGuards. BUG=none TEST=nacl_integration Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=150579

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: 3 (0 generated)
Mark Seaborn
8 years, 4 months ago (2012-08-08 16:04:13 UTC) #1
bradn
LGTM On Wed, Aug 8, 2012 at 9:04 AM, <mseaborn@chromium.org> wrote: > Reviewers: bradn, > ...
8 years, 4 months ago (2012-08-08 17:04:01 UTC) #2
bradn
8 years, 4 months ago (2012-08-08 17:04:03 UTC) #3
LGTM


On Wed, Aug 8, 2012 at 9:04 AM, <mseaborn@chromium.org> wrote:

> Reviewers: bradn,
>
> Description:
> NaCl: Update revision in DEPS, r9391 -> r9403
>
> This pulls in the following Native Client changes:
>
> r9392: (khim) Speedup the check-decoder test
> r9393: (shcherbina) Validator_ragel: fix validator_test segfault on no
> input
> files
> r9394: (bradchen) Restrict underdefined bit fields for lfence, mfence and
> sfence.
> r9395: (mseaborn) Debug stub: Make HLT produce the same signal on Windows
> and
> Linux
> r9396: (arbenson) Allocate zero-based memory if prereserved memory found on
> x86-64
> r9397: (robertm) Make a function at a time translation the default for
> pnacl.
> r9398: (bradchen) Minor clean to enuminst; fixes a base register analysis
> error.
> r9399: (robertm) Increase test coverage on pnacl bot.
> r9400: (bradchen) Fix ragel checking of unsandboxed uses of RSI/RDI and
> RBP/RSP.
> r9401: (phosek) SConstruct fix for OS X 10.8 and newer which deprecated and
> removed older OS X SDKs.
> r9402: (petarj)     [MIPS] Toolchain scripts for MIPS architecture.
> r9403: (arbenson) Pull out NaClVmmapAdd calls in NaClMprotectGuards.
>
> BUG=none
> TEST=nacl_integration
>
>
> Please review this at
http://codereview.chromium.**org/10837160/<http://codereview.chromium.org/108...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>   M DEPS
>
>
> Index: DEPS
> diff --git a/DEPS b/DEPS
> index 99cd255603e9aa772535dbcc781d06**7fc28f4315..**
> 27c81f191f06e27ba31eacf44e0723**39db588060 100644
> --- a/DEPS
> +++ b/DEPS
> @@ -12,7 +12,7 @@ vars = {
>    "chromium_git": "http://git.chromium.org/git",
>    "chromiumos_git":
"http://git.chromium.org/**chromiumos<http://git.chromium.org/chromiumos>
> ",
>    "swig_revision": "69281",
> -  "nacl_revision": "9391",
> +  "nacl_revision": "9403",
>    # After changing nacl_revision, run 'glient sync' and check
> native_client/DEPS
>    # to update other nacl_*_revision's.
>    "nacl_tools_revision": "9330",  # native_client/DEPS: tools_rev
>
>
>

Powered by Google App Engine
This is Rietveld 408576698