[PATCH] D69616: [FPEnv] The inliner shouldn't mix strictfp and non-strictfp functions.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 30 11:21:59 PDT 2019


craig.topper added inline comments.


================
Comment at: llvm/lib/Analysis/InlineCost.cpp:2039
+  // overrides the AlwaysInline attribute!
+  // This should be improved at some point.
+  if (Call.getCaller()->hasFnAttribute(Attribute::StrictFP) !=
----------------
Should this be a FIXME or TODO so it stands out?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69616





More information about the llvm-commits mailing list