On Tue, 11 Sep 2018, Qing Zhao wrote: > no immediate use case for -finline-visibility=extern right now. But I guess > that GCC developers might use this option to control inlining scope for > debugging or development purpose in the future.
In that case I'd recommend to simplify the patch by implementing only the part to suppress inlining of non-static functions. No need to overengineer. > thanks for the suggestion, how about -finline-linkage=[all|extern|static]? I would suggest -finline-only-static. Note however that I do not have the authority to review this: you still need an opinion from an appropriate maintainer. Alexander