chandlerc added a comment.

Just to be explicit, I agree with Hal's summary. This seems like the right 
engineering tradeoff and I don't find anything particularly unsatisfying about 
it.

In https://reviews.llvm.org/D28404#675616, @mehdi_amini wrote:

> Also note that @chandlerc in r290398 made clang adding "noinline" on every 
> function at O0 by default, which seems very similar to what I'm doing here.
>
> We're still waiting for @rsmith to comment whether it'd be better to `have a 
> LangOpts flag that basically means "pragma clang optimize off is always in 
> effect."` and `Have Sema pretend the pragma is in effect at all times, at 
> -O0`.


FWIW, I have no real opinion about this side of it, I see it more as a detail 
of how Clang wants to implement this kind of thing.


https://reviews.llvm.org/D28404



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to