<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<p>On 14/04/2020 09:02, Chris Johns wrote:<br>
</p>
<blockquote type="cite"
cite="mid:48a01510-f43b-df01-12b4-5018e2eff154@rtems.org">On
14/4/20 4:57 pm, Sebastian Huber wrote:
<br>
<blockquote type="cite" style="color: #000000;">Close #3944.
<br>
---
<br>
rtems/config/rtems-bsp.cfg | 2 +-
<br>
1 file changed, 1 insertion(+), 1 deletion(-)
<br>
<br>
diff --git a/rtems/config/rtems-bsp.cfg
b/rtems/config/rtems-bsp.cfg
<br>
index d50615e..91296b1 100644
<br>
--- a/rtems/config/rtems-bsp.cfg
<br>
+++ b/rtems/config/rtems-bsp.cfg
<br>
@@ -231,7 +231,7 @@
<br>
# The linker flags need to include the machine options for the
BSP.
<br>
#
<br>
%define mflags_filter awk '{for (i=1;i<NF;++i) if ($i ~
/\-m/) printf("%s ",$i);}'
<br>
-%define rtems_bsp_ldflags %{rtems_bsp_ldflags} $(echo
"%{rtems_bsp_cflags}" | %{mflags_filter})
<br>
+%define rtems_bsp_ldflags %{rtems_bsp_ldflags} $(echo
"%{rtems_bsp_cflags}" | %{mflags_filter}) -Wl,--gc-sections
<br>
</blockquote>
<br>
What about adding a define like ..
<br>
<br>
%define rtems_bsp_ldflags_extra -Wl,--gc-sections
<br>
<br>
.. or something like that and then adding it to rtems_bsp_ldflags?
<br>
<br>
It would make it easier to track.
</blockquote>
Could you please change the patch and push it. In RTEMS 6, we should
replace this flag tinkering.<br>
</body>
</html>