[PATCH] D142567: DAG: Remove hasBitPreservingFPLogic

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 25 10:54:36 PST 2023


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

I think hasBitPreservingFPLogic() is an artifact of the old days when FNEG/FABS would sometimes get expanded using FSUB etc. If we don't do that anymore, then sure, let's kill the hook off.  LGTM.



================
Comment at: llvm/lib/Target/SystemZ/SystemZISelLowering.h:1
+
 //===-- SystemZISelLowering.h - SystemZ DAG lowering interface --*- C++ -*-===//
----------------
Accidental whitespace?


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

https://reviews.llvm.org/D142567



More information about the llvm-commits mailing list