[PATCH] D79916: Map -O to -O1 instead of -O2

Brandon Bergren via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 17 11:29:34 PDT 2020


Bdragon28 added a comment.

In D79916#2279884 <https://reviews.llvm.org/D79916#2279884>, @jrtc27 wrote:

> But also you really should not get warnings for unused static functions in included headers, only ones defined in the C source file itself. We'd have countless warnings in the kernel across all architectures otherwise.

I agree. But that's what it is doing when using always_inline in combination with -Wunused-function.

There is currently no real usage of always_inline in system headers though, so maybe I'm just the first to complain about it?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79916/new/

https://reviews.llvm.org/D79916



More information about the cfe-commits mailing list