[libc-commits] [PATCH] D145584: [libc] Add support for setjmp and longjmp in riscv

[Alex Bradbury via Phabricator via libc-commits]

asb added inline comments.


================
Comment at: libc/src/setjmp/longjmp.cpp:79
+
+#if __riscv_float_abi_double
+  register double fs0 __asm__("fs0");
----------------
This is going to fail to reload FP state for `__riscv_float_abi_single`. If it's not going to be supported in this version of the patch, perhaps best to `#error`? (and do the same in setjmp).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145584