[PATCH] D71741: Add size of FP environment to DataLayout

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Do we really need this in the datalayout?  I can imagine certain frontends might care, but I can't imagine optimizations inserting new save/restore operations.

> As an example, inlining a function that requires default FP environment into a function where FP is non-standard requires saving the FP environment before entry into the inlined function and resoring it upon return.

Not sure what the inliner has to do with this; the required FP environment for each floating-point operation doesn't change if you inline a function.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71741