Hi Kewen,
> On 28 Sep 2022, at 06:30, Kewen.Lin via Gcc-patches <[email protected]>
> wrote:
> PR106680 shows that -m32 -mpowerpc64 is different from
> -mpowerpc64 -m32, this is determined by the way how we
> handle option powerpc64 in rs6000_handle_option.
>
> Segher pointed out this difference should be taken as
> a bug and we should ensure that option powerpc64 is
> independent of -m32/-m64. So this patch removes the
> handlings in rs6000_handle_option and add some necessary
> supports in rs6000_option_override_internal instead.
>
> With this patch, if users specify -m{no-,}powerpc64, the
> specified value is honoured, otherwise, for 64bit it
> always enables OPTION_MASK_POWERPC64 while for 32bit
> it disables OPTION_MASK_POWERPC64 if OS_MISSING_POWERPC64.
>
> Bootstrapped and regress-tested on:
> - powerpc64-linux-gnu P7 and P8 {-m64,-m32}
> - powerpc64le-linux-gnu P9 and P10
> - powerpc-ibm-aix7.2.0.0 {-maix64,-maix32}
>
> Hi Iain, could you help to test this on darwin to ensure
> it won't break darwin's build and new tests are fine?
> Thanks in advance!
Will do, it will take a day or so, thanks,
Iain
