[cfe-commits] [patch] Alternative fix for pr9614

Rafael Ávila de Espíndola rafael.espindola at gmail.com
Fri Oct 28 13:24:41 PDT 2011


>> Please keep the CodeGenOpts.OptimizationLevel test first, and put the
>> isTriviallyRecursiveViaAsm test at the end.
>
> I did that, but it changes the meaning for always_inline functions. We
> can always revisit this if one is found in the wild.
>

cartman found one:

extern int foo_alias (void) __asm__ ("foo");
inline __attribute__ ((__always_inline__))  int foo (void) {
   return foo_alias ();
}
int bar() {
   foo();
}

We must not output an available_externally in this case. I will try to 
fix it.

Cheers,
Rafael



More information about the cfe-commits mailing list