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

Issue 563303003: Subzero: Always be sure x87 FP stack is emptied after a call. (Closed)

Created:
6 years, 3 months ago by Jim Stichnoth
Modified:
6 years, 3 months ago
CC:
native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Visibility:
Public.

Description

Subzero: Always be sure x87 FP stack is emptied after a call. In x86-32, floating point values are returned to the caller on the top of the x87 floating point stack. The caller is required to remove it from the x87 FP stack, e.g. via the fstp instruction. This must be done even when the return value is not actually used anywhere else in the function, in which case O2 is likely to want to dead-code eliminate the fstp instruction. We enforce this by adding a fake use of the fstp destination. BUG= none R=jvoung@chromium.org Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=a522972

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+64 lines, -1 line) Patch
M src/IceTargetLoweringX8632.cpp View 1 chunk +4 lines, -1 line 0 comments Download
A tests_lit/llvm2ice_tests/fpcall.ll View 1 chunk +60 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
Jim Stichnoth
This fix allows 253.perlbmk to run under O2.
6 years, 3 months ago (2014-09-12 19:24:18 UTC) #2
jvoung (off chromium)
lgtm
6 years, 3 months ago (2014-09-12 19:44:01 UTC) #3
Jim Stichnoth
6 years, 3 months ago (2014-09-12 20:06:14 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as a522972 (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698