Re: RFR: 8204541: Correctly support AIX xlC 16.1 symbol visibility flags [v2]

2022-03-19 Thread Kim Barrett
> On Mar 18, 2022, at 4:11 AM, Thomas Stuefe wrote: > > On Fri, 18 Mar 2022 08:03:11 GMT, Ichiroh Takiguchi > wrote: > >>> This change is just for AIX platform with IBM XL C/C++ 16.1. >>> By this change, lib's entry points are reduced by compilation and linkage >>> option changes. >>> >>> Wi

Re: RFR: 8204541: Correctly support AIX xlC 16.1 symbol visibility flags [v2]

2022-03-18 Thread Thomas Stuefe
On Fri, 18 Mar 2022 08:03:11 GMT, Ichiroh Takiguchi wrote: >> This change is just for AIX platform with IBM XL C/C++ 16.1. >> By this change, lib's entry points are reduced by compilation and linkage >> option changes. >> >> Without changes: >> >> $ dump -X32_64 -Tv lib/libawt.so | grep EXP |

Re: RFR: 8204541: Correctly support AIX xlC 16.1 symbol visibility flags [v2]

2022-03-18 Thread Tyler Steele
On Fri, 18 Mar 2022 08:03:11 GMT, Ichiroh Takiguchi wrote: >> This change is just for AIX platform with IBM XL C/C++ 16.1. >> By this change, lib's entry points are reduced by compilation and linkage >> option changes. >> >> Without changes: >> >> $ dump -X32_64 -Tv lib/libawt.so | grep EXP |

Re: RFR: 8204541: Correctly support AIX xlC 16.1 symbol visibility flags [v2]

2022-03-18 Thread Magnus Ihse Bursie
On Fri, 18 Mar 2022 08:03:11 GMT, Ichiroh Takiguchi wrote: >> This change is just for AIX platform with IBM XL C/C++ 16.1. >> By this change, lib's entry points are reduced by compilation and linkage >> option changes. >> >> Without changes: >> >> $ dump -X32_64 -Tv lib/libawt.so | grep EXP |

Re: RFR: 8204541: Correctly support AIX xlC 16.1 symbol visibility flags [v2]

2022-03-18 Thread Thomas Stuefe
On Fri, 18 Mar 2022 08:07:53 GMT, Thomas Stuefe wrote: >> Ichiroh Takiguchi has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Remove unexpected comment > > So, does this mean we cannot build with older xlC versions anymore? Just > curious

Re: RFR: 8204541: Correctly support AIX xlC 16.1 symbol visibility flags [v2]

2022-03-18 Thread Ichiroh Takiguchi
On Fri, 18 Mar 2022 08:07:53 GMT, Thomas Stuefe wrote: >> Ichiroh Takiguchi has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Remove unexpected comment > > So, does this mean we cannot build with older xlC versions anymore? Just > curious

Re: RFR: 8204541: Correctly support AIX xlC 16.1 symbol visibility flags [v2]

2022-03-18 Thread Thomas Stuefe
On Fri, 18 Mar 2022 08:03:11 GMT, Ichiroh Takiguchi wrote: >> This change is just for AIX platform with IBM XL C/C++ 16.1. >> By this change, lib's entry points are reduced by compilation and linkage >> option changes. >> >> Without changes: >> >> $ dump -X32_64 -Tv lib/libawt.so | grep EXP |

Re: RFR: 8204541: Correctly support AIX xlC 16.1 symbol visibility flags [v2]

2022-03-18 Thread Ichiroh Takiguchi
> This change is just for AIX platform with IBM XL C/C++ 16.1. > By this change, lib's entry points are reduced by compilation and linkage > option changes. > > Without changes: > > $ dump -X32_64 -Tv lib/libawt.so | grep EXP | wc > 914 > > With changes > > $ dump -X32_64 -Tv lib/libawt.so | g

Re: RFR: 8204541: Correctly support AIX xlC 16.1 symbol visibility flags [v2]

2022-03-18 Thread Ichiroh Takiguchi
On Fri, 18 Mar 2022 07:41:28 GMT, Magnus Ihse Bursie wrote: >> Ichiroh Takiguchi has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Remove unexpected comment > > make/autoconf/flags-ldflags.m4 line 80: > >> 78: >> 79: elif test "x$TOOLC