[PATCH] D51646: DAG: Preserve FMF when creating fminnum/fmaxnum
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 5 09:12:51 PDT 2018
spatel added a reviewer: efriedma.
spatel added a comment.
Is this just about 'nnan' behavior? If so, can we just use/expand the existing SelectPatternNaNBehavior?
IOW, why is 'nsz' relevant?
Eli raised a question about 'nsz' on fcmp:
https://bugs.llvm.org/show_bug.cgi?id=38086
Does the motivation for this patch make any of the proposed fixes more or less appealing?
Regardless to any answers to the above, I think we should split this into separate IR/DAG patches with tests for each side.
https://reviews.llvm.org/D51646
More information about the llvm-commits
mailing list