• bmeurer's avatar
    [turbofan] Change Float64Max/Float64Min to JavaScript semantics. · ba092fb0
    bmeurer authored
    So far we don't have a useful way to inline Math.max or Math.min in
    TurboFan optimized code. This adds new operators NumberMax and NumberMin
    and changes the Float64Max/Float64Min operators to have JavaScript
    semantics instead of the C++ semantics that it had previously.
    
    This also removes support for recognizing the tenary case in the
    CommonOperatorReducer, since that doesn't seem to have any positive
    impact (and actually doesn't show up in regular JavaScript, where
    people use Math.max/Math.min instead).
    
    Drive-by-fix: Also nuke the unused Float32Max/Float32Min operators.
    
    R=jarin@chromium.org
    
    Review-Url: https://codereview.chromium.org/2170343002
    Cr-Commit-Position: refs/heads/master@{#37971}
    ba092fb0
common-operator-reducer-unittest.cc 16.1 KB