labath added inline comments.
================
Comment at: lldb/test/API/lang/c/fpeval/main.c:3-4
+{
+ double a = 42.0;
+ double b = 2.0;
+ return 0; //// Set break point at this line.
----------------
kpdev42 wrote:
> clayborg wrote:
> > We should be testing "float" and "long double" as well to verify those work.
> Long double is not supported by this patch, but can be added. However long
> double is platform dependent type, so it makes no sense at all comparing JIT
> and interpreted results
I think that was Greg's point. Interpreted and JITted results should provide
the same results even for platform dependent types. llvm's APFloat should know
enough about the individual types in order to emulate the operations correctly,
and lldb should know enough about the target in order to create the appropriate
APFloat instance.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126359/new/
https://reviews.llvm.org/D126359
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
