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

Issue 8366024: Add pnacl-gcc generated code back into the callingconv test mix (Closed)

Created:
9 years, 2 months ago by pdox
Modified:
9 years, 2 months ago
CC:
native-client-reviews_googlegroups.com
Visibility:
Public.

Description

Test intermixing pnacl-clang, pnacl-gcc, and nacl-gcc generated code in the callingconv test. Although a large part of the callingconv test remains disabled due to known ABI compatibility problems, eventually this test will help detect and prevent ABI changes. Committed: http://src.chromium.org/viewvc/native_client?view=rev&revision=6999

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+59 lines, -32 lines) Patch
M site_scons/site_tools/naclsdk.py View 2 chunks +14 lines, -0 lines 0 comments Download
M tests/callingconv/callingconv.c View 1 chunk +12 lines, -8 lines 0 comments Download
M tests/callingconv/generate.py View 1 chunk +6 lines, -4 lines 0 comments Download
M tests/callingconv/nacl.scons View 4 chunks +27 lines, -20 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
pdox
PTAL
9 years, 2 months ago (2011-10-20 23:41:00 UTC) #1
jvoung - send to chromium...
Question: Are there incompatibilities between pnacl-gcc and pnacl-clang too? Otherwise, the changes LGTM
9 years, 2 months ago (2011-10-21 00:06:12 UTC) #2
pdox
9 years, 2 months ago (2011-10-21 00:07:46 UTC) #3
So far, there have been no incompatibilities.

Powered by Google App Engine
This is Rietveld 408576698