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

Issue 14774009: Update PNaCl intrinsic whitelist test for moving {frame,return}address to blacklist. (Closed)

Created:
7 years, 7 months ago by jvoung (off chromium)
Modified:
7 years, 7 months ago
Reviewers:
Derek Schuff, eliben
CC:
native-client-reviews_googlegroups.com
Base URL:
http://git.chromium.org/native_client/pnacl-llvm.git@master
Visibility:
Public.

Description

Update PNaCl intrinsic whitelist test for moving {frame,return}address to blacklist. Forgot to do this in last CL: https://codereview.chromium.org/14657017/ BUG=https://code.google.com/p/nativeclient/issues/detail?id=3378 R=dschuff@chromium.org Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-llvm.git;a=commit;h=442f311

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -2 lines) Patch
M test/NaCl/PNaClABI/intrinsics.ll View 1 chunk +0 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
jvoung (off chromium)
7 years, 7 months ago (2013-05-14 00:29:23 UTC) #1
Derek Schuff
maybe clarify the subject a little (pnacl whitelist test -> pnacl intrinsic whitelist test or ...
7 years, 7 months ago (2013-05-14 00:41:43 UTC) #2
Derek Schuff
On 2013/05/14 00:41:43, Derek Schuff wrote: > maybe clarify the subject a little (pnacl whitelist ...
7 years, 7 months ago (2013-05-14 00:41:49 UTC) #3
jvoung (off chromium)
7 years, 7 months ago (2013-05-14 16:29:54 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r442f311 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698