Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 484840 - Add QA warning for -Waggressive-loop-optimizations
Summary: Add QA warning for -Waggressive-loop-optimizations
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Enhancement/Feature Requests (show other bugs)
Hardware: All Linux
: Normal QA (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2013-09-14 06:58 UTC by Ryan Hill (RETIRED)
Modified: 2013-09-15 11:28 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
portage-2.2.3-qa-aggressive-loop.patch (portage-2.2.3-qa-aggressive-loop.patch,435 bytes, patch)
2013-09-14 06:58 UTC, Ryan Hill (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Hill (RETIRED) gentoo-dev 2013-09-14 06:58:48 UTC
Created attachment 358616 [details, diff]
portage-2.2.3-qa-aggressive-loop.patch

Just ran into an infinite loop bug in a package with gcc 4.8.  We should flag these warnings (enabled by -Wall) to catch problems like this which will usually only appear at runtime.

linux/gl_draw.c: In function 'gl_drawwin':
cc1: warning: iteration 222u invokes undefined behavior [-Waggressive-loop-optimizations]
linux/gl_draw.c:329:5: note: containing loop
     for (i = 0; i < 224; i++)
     ^

This one kind of sucks because the filename/line won't be in the qawarn but it's better than nothing.
Comment 2 Zac Medico gentoo-dev 2013-09-15 11:28:02 UTC
This is fixed in 2.2.5.