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

Issue 1367403004: Subzero: Add missing content to CfgLocalAllocator. (Closed)

Created:
5 years, 2 months ago by Jim Stichnoth
Modified:
5 years, 2 months ago
Reviewers:
Karl, John
CC:
native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Subzero: Add missing content to CfgLocalAllocator. The std::list<> implementation used by g++ needs some extra stuff defined in the custom allocator. This can be smoke-tested with: make -f Makefile.standalone CXX=g++ LLVM_EXTRA_WARNINGS="-Wno-unknown-pragmas -Wno-unused-parameter -Wno-comment -Wno-enum-compare -Wno-strict-aliasing" STDLIB_FLAGS= until the link fails for unrelated reasons. BUG= https://code.google.com/p/nativeclient/issues/detail?id=4325 R=kschimpf@google.com Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=91d1b80f69aa32922518606718753f5decdcb546

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -2 lines) Patch
M src/IceDefs.h View 1 chunk +12 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
Jim Stichnoth
5 years, 2 months ago (2015-10-01 17:33:41 UTC) #2
Karl
lgtm
5 years, 2 months ago (2015-10-01 18:33:50 UTC) #3
Jim Stichnoth
5 years, 2 months ago (2015-10-01 20:24:38 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
91d1b80f69aa32922518606718753f5decdcb546 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698