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. >>>>>> >>>>>
