Skip to content

Commit

Permalink
Add upcoming_changes note
Browse files Browse the repository at this point in the history
  • Loading branch information
hoodmane authored and charris committed Mar 31, 2022
1 parent f2119f9 commit f1c1014
Showing 1 changed file with 23 additions and 0 deletions.
23 changes: 23 additions & 0 deletions doc/release/upcoming_changes/21154.improvement.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
Math C library feature detection now works for wasm
---------------------------------------------------
The previous feature detection generated C stubs like:

.. code-block:: C
void exp(void); // incorrect signature!
exp();
Then checked whether compiling and linking these worked. This generates warnings
that complain that ``exp`` actually has signature ``double exp(double)`` but would
otherwise work. However ``wasm-ld`` creates a hard error in this case, so when
compiling to a wasm target everything is detected as missing.

This is now fixed: we feature detect instead with code like:

.. code-block:: C
double exp(double); // correct signature!
exp((double)0);
which ``wasm-ld`` will link. As a side effect, the feature detection code now
works without generating linker warnings.

0 comments on commit f1c1014

Please sign in to comment.