[PATCH] D71742: Added intrinsics for access to FP environment

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 23 09:06:42 PST 2019


arsenm added inline comments.


================
Comment at: llvm/docs/LangRef.rst:16829
+
+      declare void @llvm.get.fenv(i8* <val>)
+
----------------
Why a pointer? Why not an i64 argument? The lowering of this through memory seems problematic for any target that doesn't implement this as the fe libcalls


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71742





More information about the llvm-commits mailing list