HomePhabricator

[x86] explicitly set cost of integer add/sub

Description

[x86] explicitly set cost of integer add/sub

There are no test changes here in the existing cost model
regression tests because integer add/sub have a default
legal cost of 1 already. This would break, however, if
we custom lower those ops because the default cost model
assumes that custom-lowered ops are more expensive.

This is similar to the change in rL350403. See discussion
in D56011 for more details. When we enhance that patch to
handle integer ops, we need this cost model change to avoid
unintended diffs here from the custom lowering.

Details

Committed
spatelJan 6 2019, 8:21 AM
Parents
rL350495: Fix bug in test found by the diagnostic added in r350340.
Branches
Unknown
Tags
Unknown