[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