HomePhabricator

[SelectionDAG] Do minnum->minimum at legalization time instead of building time

Authored by bkramer on Jul 1 2019, 4:00 AM.

Description

[SelectionDAG] Do minnum->minimum at legalization time instead of building time

The SDAGBuilder behavior stems from the days when we didn't have fast
math flags available in SDAG. We do now and doing the transformation in
the legalizer has the advantage that it also works for vector types.

llvm-svn: 364743

Details

Auditors
tlively
Committed
bkramerJul 1 2019, 4:00 AM
Parents
rGd74f2d0a8609: [benchmark] Disable CMake get_git_version
Branches
Unknown
Tags
Unknown

Event Timeline

aheejin added a subscriber: tlively.Jul 2 2019, 8:44 PM

@tlively Just wanted to check if this is OK for us.

tlively accepted this commit.Jul 3 2019, 11:29 AM

Yeah this looks great. Thanks!

All concerns with this commit have now been addressed.Jul 3 2019, 11:29 AM