Re: PR84229 part 1: Avoid cloning of functions that calls va_arg_pack

2018-02-21 Thread Jakub Jelinek
On Wed, Feb 21, 2018 at 08:09:28PM +0100, Jan Hubicka wrote: > --- ipa-cp.c (revision 257844) > +++ ipa-cp.c (working copy) > @@ -630,6 +630,24 @@ determine_versionability (struct cgraph_ >reason = "calls comdat-local function"; > } > > + /* Functions calling BUILT_IN_VA_ARG_PACK

Re: PR84229 part 1: Avoid cloning of functions that calls va_arg_pack

2018-02-21 Thread Jan Hubicka
> On Wed, Feb 21, 2018 at 08:09:28PM +0100, Jan Hubicka wrote: > > --- ipa-cp.c(revision 257844) > > +++ ipa-cp.c(working copy) > > @@ -630,6 +630,24 @@ determine_versionability (struct cgraph_ > >reason = "calls comdat-local function"; > > } > > > > + /* Functions c

PR84229 part 1: Avoid cloning of functions that calls va_arg_pack

2018-02-21 Thread Jan Hubicka
Hi, this fixes first part of the issue in PR84229. The actual testcase dies because ipa-cp decides to cone function calling va_arg_pack which is later not inlined. Such functions works only when they are inlined and thus it makes no sense to inline them. I disabled cloning only for external functi