• zhengxing.li's avatar
    X87: [turbofan] Change Float64Max/Float64Min to JavaScript semantics. · 691e6d39
    zhengxing.li authored
      port ba092fb0 (r37971)
    
      original commit message:
      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).
    
    BUG=
    
      Drive-by-fix: Also nuke the unused Float32Max/Float32Min operators.
    Review-Url: https://codereview.chromium.org/2187463005
    
    Cr-Commit-Position: refs/heads/master@{#38119}
    691e6d39
code-generator-x87.cc 89.4 KB