Closed
Bug 1149416
Opened 10 years ago
Closed 10 years ago
nsTArray.h:188:44: error: attempt to free a non-heap object 'tautoconst' [-Werror=free-nonheap-object]
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox40 fixed)
RESOLVED
FIXED
mozilla40
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(1 file)
(deleted),
patch
|
cpeterson
:
review+
|
Details | Diff | Splinter Review |
Bug 1138293 replaced a bunch of moz_free with free, and as a result, this triggered a warning in GCC that only happens under some particular circumstances (that particular one only happens on pgo linux64 builds, for instance).
Cf. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54202 it appears this warning is not accurate and may have plenty of false positives. We were getting away with it because we weren't using free() that much.
I propose that we disable this warning, on PGO builds only for now.
Assignee | ||
Comment 1•10 years ago
|
||
Making this a r=me for landing, but hoping cpeterson will regularize.
Attachment #8585920 -
Flags: review?(cpeterson)
Assignee | ||
Comment 2•10 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #1)
> Created attachment 8585920 [details] [diff] [review]
> Don't make free-nonheap-object warning an error
>
> Making this a r=me for landing, but hoping cpeterson will regularize.
To make things clearer, this needs landing to unbust linux64 pgo builds after bug 1138293.
Assignee | ||
Comment 3•10 years ago
|
||
Comment 4•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Comment 5•10 years ago
|
||
Comment on attachment 8585920 [details] [diff] [review]
Don't make free-nonheap-object warning an error
Review of attachment 8585920 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM
Attachment #8585920 -
Flags: review?(cpeterson) → review+
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•