Hi,

Gentle ping this patch:

https://gcc.gnu.org/pipermail/gcc-patches/2021-September/578552.html

One related patch [1] is ready to commit, whose test cases rely on
this patch if no changes are applied to them.

[1] https://gcc.gnu.org/pipermail/gcc-patches/2021-September/579658.html

BR,
Kewen

>>> on 2021/9/1 下午2:55, Kewen.Lin via Gcc-patches wrote:
>>>> Hi!
>>>>
>>>> This patch is to fix the inconsistent behaviors for non-LTO mode
>>>> and LTO mode.  As Martin pointed out, currently the function
>>>> rs6000_can_inline_p simply makes it inlinable if callee_tree is
>>>> NULL, but it's wrong, we should use the command line options
>>>> from target_option_default_node as default.  It also replaces
>>>> rs6000_isa_flags with the one from target_option_default_node
>>>> when caller_tree is NULL as rs6000_isa_flags could probably
>>>> change since initialization.
>>>>
>>>> It also extends the scope of the check for the case that callee
>>>> has explicit set options, for test case pr102059-2.c inlining can
>>>> happen unexpectedly before, it's fixed accordingly.
>>>>
>>>> As Richi/Mike pointed out, some tuning flags like MASK_P8_FUSION
>>>> can be neglected for inlining, this patch also exludes them when
>>>> the callee is attributed by always_inline.
>>>>
>>>> Bootstrapped and regtested on powerpc64le-linux-gnu Power9.
>>>>
>>>> BR,
>>>> Kewen
>>>> -----
>>>> gcc/ChangeLog:
>>>>
>>>>    PR ipa/102059
>>>>    * config/rs6000/rs6000.c (rs6000_can_inline_p): Adjust with
>>>>    target_option_default_node and consider always_inline_safe flags.
>>>>
>>>> gcc/testsuite/ChangeLog:
>>>>
>>>>    PR ipa/102059
>>>>    * gcc.target/powerpc/pr102059-1.c: New test.
>>>>    * gcc.target/powerpc/pr102059-2.c: New test.
>>>>    * gcc.target/powerpc/pr102059-3.c: New test.
>>>>    * gcc.target/powerpc/pr102059-4.c: New test.
>>>>
>>>

Reply via email to