[PATCH] D106191: [clang] Option control afn flag

[Qiu Chaofan via Phabricator via cfe-commits]

qiucf added a comment.

Making a new option mapped to another float op flag looks reasonable, but is there any clearer motivation for this? (such as the need for `-Ofast -fno-approx-func`)



================
Comment at: llvm/include/llvm/Target/TargetOptions.h:179
+    /// with approximate calculations
+    unsigned ApproxFuncFPMath : 1;
+
----------------
`-enable-no-signed-zeros-fp-math` is an `llc` flag.

Do we really have `-enable-approx-func-fp-math` for `llc` now?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106191