[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