RFR: JDK-8329257: AIX: Switch HOTSPOT_TOOLCHAIN_TYPE from xlc to gcc

[Joachim Kern]

On Tue, 2 Apr 2024 09:14:10 GMT, Joachim Kern <jkern at openjdk.org> wrote:

>> Other than that, and kind of depending on your answer: How important is it that we catch every use of the original malloc? Can be safely mix the original malloc with vec_malloc if logging is not involved?
>> 
>> I am asking, because from that it depends whether this hunk needs to appear right behind `#include <stdlib.h>` or whether we can move it into the middle of the file together with the other AIX stuff. 
>> 
>> Because, if we move it into the middle of the file, we may miss any uses of malloc that may happen in system headers (would be unusual for that to happen but with IBM one never knows).
>
> Hi Thomas,
> I would like to get totally rid of this, because as I mentioned IBM already modified the `stdlib.h` header not using `#define malloc vec_malloc` any more (and all the other vec_... defines). We have to ask the adoptium colleagues at IBM if they already have raised their build environment by the 2 SP levels needed.
> In principle we had to do the same workaround for `calloc, free,...` too, but they didn't show up as errors in the logging files.
> These lines where never meant to stay for long. Just to be able to compile until IBM fixes the issue, which is done now.

@suchismith1993
Hi Suchi, can you please tell me when you will raise your build environment from AIX 7.2 TL5 SP5 to SP7?
I' am asking you, because I want to get rid of this nasty workaround.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18536#discussion_r1547473723