On Thu, Feb 01, 2018 at 09:47:23AM +0100, Richard Biener wrote: > I wonder if you actually ran into the case of ABS_EXPR being > passed to rewrite_to_non_trapping_overflow?
I haven't. In the first vrsion of the patch I wasn't using operation_no_trapping_overflow and so just ignored ABS_EXPR, with that helper either I'd need to explicitly punt on ABS_EXPR, or implement it and chose the latter. Yes, we have no testsuite coverage for it :(. > Ok for trunk. Thanks. Jakub