On 01/24/2017 06:03 PM, Christophe Lyon wrote:
Ha... the regression occurred between r 244818 and r 244816, and I read r 244816 ChangeLog too quickly and did not notice it was modifying ifcvt.c in addition to x86-only files.So it's likely that it's your other patch for pr78634 that caused the regression I mentioned. Does it make more sense?
That's possible. That added a missing cost check, so the question becomes - is the change in generated assembly sensible, given the selected CPU type?
Bernd
