[PATCH] D148750: ValueTracking: Fix computeKnownFPClass for fabs

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 21 08:51:48 PDT 2023


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

Code LGTM.

The situation with the fabs helpers is not great. We have two, they do opposite things, one isn't documented and the other is documented in a way that I can't understand.


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

https://reviews.llvm.org/D148750



More information about the llvm-commits mailing list