> On Sep 12, 2018, at 2:46 AM, Alexander Monakov <amona...@ispras.ru> wrote: > > 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.
the current design is trying to make the functionality more general although the part of disable inlining of static functions does not have immediate use case. the additional implementation for the disabling inlining of static functions is very trivial and minimal. Personally I’d like to keep the current design. However, I am flexible on this. > >> 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. understand, thanks a lot for your comments and suggestions. thanks. Qing > > Alexander
