casaroli commented on PR #20723: URL: https://github.com/apache/nuttx/pull/20723#issuecomment-6279749196
An update on point 1: I wrote before that this PR keeps the real names and does not use `--wrap`. xiaoxiang781216 asked for `--wrap`, and it is the better way, so the PR now uses it: libc defines `__wrap___emutls_get_address()` and `__wrap___emutls_register_common()`, and the image, the modules and the kernel-build programs link with `--wrap`. The real names are weak aliases now, so in a `--whole-archive` link the strong symbols of libgcc win without a conflict, and the `--whole-archive` commit is gone. `qemu-armv7a:nsh` and `mps2-an500:nsh` with Arm GNU 13.2 and 15.3, `qemu-armv7a:nsh` with CMake and `rv-virt:nsh` link with `DEBUG_LINK_WHOLE_ARCHIVE`. Thank you for the hint about `tools/Config.mk`: the options are there. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
