[PATCH] D87571: [DAGCombiner] Fold fmin/fmax with INF

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 05:40:22 PDT 2020

Previous message: [PATCH] D87606: [ADT] Remove unnecessary copies in SmallVectorImpl::swap
Next message: [PATCH] D87571: [DAGCombiner] Fold fmin/fmax with INF
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

spatel accepted this revision.
spatel added a comment.

Expanded patch still LGTM.
Side note: change "isLargest" name to "isLargestMagnitude" to make that API less confusing?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87571/new/

https://reviews.llvm.org/D87571

Previous message: [PATCH] D87606: [ADT] Remove unnecessary copies in SmallVectorImpl::swap
Next message: [PATCH] D87571: [DAGCombiner] Fold fmin/fmax with INF
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list