RFR: 8348286: [AIX] clang 17 introduces new warning Wtentative-Definitions which produces Build errors [v3]

Joachim Kern jkern at openjdk.org
Fri Jan 24 09:57:48 UTC 2025


On Thu, 23 Jan 2025 20:31:03 GMT, Phil Race <prr at openjdk.org> wrote:

>> Joachim Kern has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   remove old solution
>
> If absolutely nothing else needs fp_pipewire.h I guess that ought to be OK ... but this is very new code.
> I'd like @azvegint to look at this as he may have had some reason that isn't obvious.
> 
> I don't follow why you found it necessary to remove the include statement from the .c file in the image case ?
> And I agree that there was actually nothing wrong with the existing code. The compiler must have been confused.
> 
> And other than  *building* what testing was done with these changes ?

@prrace Is this OK for you now after Aleksandr has tested and approved the PR?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/23236#issuecomment-2612112103


More information about the build-dev mailing list