Skip to content

ci: detect more warnings via `-Og`

Patrick Steinhardt git 4 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v4 Latest2 patches 4 repliesIn review
  2. v34 patches 7 repliesSuperseded
  3. v22 patches 7 repliesSuperseded
Earlier versions · 1
VersionPostedPatchesRepliesStatus
v12 patches9 repliesSuperseded

Changes in v4 from the cover letter

- Remove the first patch as Junio has moved it into a separate topic so that it can be fast-tracked.

- Drop the patch that changes CI jobs to compile with V=1.

- Drop the override statement and instead implement CFLAGS_APPEND and LDFLAGS_APPEND via ALL_CFLAGS and ALL_LDFLAGS, respectively, as suggested by Peff.

Patches in v4 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2Makefile: add ability to append to CFLAGS and LDFLAGSMakefile+2 -20 replies
2/2ci: compile "linux-gcc-default" job with -Ogci/run-build-and-tests.sh+9 -011 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v4 yet.

Read the v4 thread

Cover letter [PATCH v4 0/2] ci: detect more warnings via `-Og`

Hi,

this is the fourth version of my patch series that starts to compile with -Og in one of our CI jobs. This is done to surface more warnings from the compiler that might not show with other optimization levels.

Changes compared to v3:

- Remove the first patch as Junio has moved it into a separate topic so that it can be fast-tracked.

- Drop the patch that changes CI jobs to compile with V=1.

- Drop the override statement and instead implement CFLAGS_APPEND and LDFLAGS_APPEND via ALL_CFLAGS and ALL_LDFLAGS, respectively, as suggested by Peff.

The range diff is not all that useful for this version, and the patch series is tiny, so I dropped it.

Thanks!

Patrick

Patrick Steinhardt (2):
Makefile: add ability to append to CFLAGS and LDFLAGS
ci: compile "linux-gcc-default" job with -Og

 Makefile                  | 4 ++--
 ci/run-build-and-tests.sh | 9 +++++++++
 2 files changed, 11 insertions(+), 2 deletions(-)
Signature · 1 line
2.45.2.436.gcd77e87115.dirty