[PATCH] gcc compatibility: silent -finline-limit=1234 & -finline-limit

Alp Toker alp at nuanti.com
Sat Jul 5 15:32:40 PDT 2014


On 05/07/2014 23:22, Joerg Sonnenberger wrote:
> On Sat, Jul 05, 2014 at 01:07:35AM +0300, Alp Toker wrote:
>> 3) Add the necessary check at that point in LLVM to disable inlining
>> if the size is greater than the flag value.
> The problem here is that GCC very likely uses a completely different
> metric, so using the same option is just likely to give unexpected
> results.

Yeah. With most backend flags/diagnostics, the best we can do is 
preserve the semantics rather than bug-for-bug compatibility.

So I don't mind if someone wants to have a shot at implementing this, 
but I'd be against silently ignoring it.

Alp.



>
> Joerg
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-- 
http://www.nuanti.com
the browser experts




More information about the cfe-commits mailing list