RFR: 8303082 : [AIX] Missing C++ name demangling with XLClang++ [v3]
Deepa Kumari
duke at openjdk.org
Thu Mar 30 18:59:24 UTC 2023
On Thu, 30 Mar 2023 10:17:36 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:
>> Deepa Kumari has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains one additional commit since the last revision:
>>
>> 8303082 : [AIX] Missing C++ name demangling with XLClang++
>
> src/hotspot/os/aix/porting_aix.cpp line 242:
>
>> 240: int status;
>> 241: char *demangled_name = abi::__cxa_demangle(p_name, nullptr, nullptr, &status);
>> 242: if ((demangled_name != nullptr) && (status == 0)) {
>
> Should the `strncpy` better depend on `status == 0` and the `free` operation on `demangled_name != nullptr`? I'm not sure about the relationship between the two checks. Maybe your version is ok, too.
Thank you so much @TheRealMDoerr and @tstuefe for highlighting this. I will tie the free to ptr!=NULL only.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/12742#discussion_r1153672262
More information about the hotspot-runtime-dev
mailing list