• bryleun's avatar
    S390: Fixed s390 simulation check for underflow in subtraction. · 0d24a0fc
    bryleun authored
    Clang optimizes away CheckOverflowForIntSub at any opt
    level (includes -O1, -O2, -O3) into a false statement,
    resulting in incorrect values being returned. As the C++
    standard considers overflows to be undefined behaviour,
    this is technically correct as compilers can assume that
    overflows never occur, but problematic in our case (where
    overflows do occur, and a specific result is expected).
    
    This change replaces the original check with a call to a
    function that is optimized in a manner that returns correct output.
    
    R=michael_dawson@ca.ibm.com,jyan@ca.ibm.com,mtbrandyberry@ca.ibm.com,joransiu@ca.ibm.com,danno@chromium.org,jkummerow@chromium.org,jochen@chromium.org
    
    BUG=
    
    Review URL: https://codereview.chromium.org/1826043002
    
    Cr-Commit-Position: refs/heads/master@{#35082}
    0d24a0fc
Name
Last commit
Last update
benchmarks Loading commit data...
build Loading commit data...
docs Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party/binutils Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.gitignore Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
ChangeLog Loading commit data...
DEPS Loading commit data...
LICENSE Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
Makefile Loading commit data...
Makefile.android Loading commit data...
Makefile.nacl Loading commit data...
OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...
snapshot_toolchain.gni Loading commit data...