[PATCH] D48085: [DAGCombiner] restrict (float)((int) f) --> ftrunc with no-signed-zeros

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 26 08:39:43 PDT 2018


lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

I still think the docs need //some// adjustment.
And not for all arches the there are full set of tests (i.e. one with `"no-signed-zeros-fp-math"="true"`, one without).
But this is a regression, so i'd guess that can be iterated later on.

It would be best if someone else could agree/disagree, but other than that, LG.


https://reviews.llvm.org/D48085





More information about the llvm-commits mailing list