Ah -- I didn't notice that patch. It looks like both do essentially the same thing.
That one identifies the use of the c++ test runner by checking for the presence of the `lang_test_file` variable, and in that case `libitm_target_compile` adds the options, while in my patch the c++ test runner adds options to `ALWAYS_CFLAGS` which `libitm_target_compile` uses directly.
On 1/3/25 17:14, Joseph Myers wrote:
External email: Use caution opening links or attachments Does this patch cover everything dealt with by <https://gcc.gnu.org/pipermail/gcc-patches/2024-December/672242.html> ([PATCH] testsuite: libitm: Remove build directory path from test names), or would some separate fix for that issue still be needed in the presence of this patch? -- Joseph S. Myers josmy...@redhat.com
