On 3/23/25 8:28 AM, Iain Sandoe wrote:
Hi Jeff,

On 23 Mar 2025, at 14:25, Jeff Law <jeffreya...@gmail.com> wrote:
On 3/23/25 8:03 AM, Iain Sandoe wrote:
Tested on x86_64-Linux, Darwin,
OK for trunk?
backports?
thanks
Iain
--- 8< ---
Actually, the issue is not local to the libitm case, it currently affects
any 'cxx=true' top-level configured target library.
The issue is a missing export of CXX_FOR_TARGET.
        PR libitm/88319
ChangeLog:
        * Makefile.in: Regenerate.
        * Makefile.tpl: Add CXX_FOR_TARGET to NORMAL_TARGET_EXPORTS.
So how does one trigger this failure?   ISTM that every libitm build should be 
failing.  I think the patch is likely fine, but struggle to see why this hasn't 
been more widely reported.

take your current build, and then delete <target>/{,mlib}/libitm
then try
“make ….. all-target-libitm”  .. I believe it will fail with a libtool error 
(because the compiler is not passed - only the options)

That was certainly failing on both x86_64 linux/darwin before the patch.
To be clear, I don't doubt it was failing, just trying to understand the failure mode. So it only pops up during a rebuild in an existing tree which explains why we haven't been seeing this reported more often.

Thanks.  Ok for the trunk.

jeff

Reply via email to