[Bug middle-end/38509] Bogus "attempt to free a non-heap object" warning

[Bug middle-end/38509] Bogus "attempt to free a non-heap object" warning

[meheff at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38509

--- Comment #4 from Mark Heffernan <meheff at gcc dot gnu.org> 2011-08-23 18:06:48 UTC ---
Author: meheff
Date: Tue Aug 23 18:06:42 2011
New Revision: 178004

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=178004
Log:
2011-08-23  Mark Heffernan  <meheff@google.com>

        PR middle-end/38509
        * common.opt (Wfree-nonheap-object): New option.
        * doc/invoke.texi (Warning options): Document -Wfree-nonheap-object.
        * builtins.c (maybe_emit_free_warning): Add OPT_Wfree_nonheap_object
        to warning.
        (expand_builtin): Make warning conditional.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/builtins.c
    trunk/gcc/common.opt
    trunk/gcc/doc/invoke.texi

[Bug middle-end/38509] Bogus "attempt to free a non-heap object" warning

[meheff at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38509

Mark Heffernan <meheff at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|missed-optimization         |
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |meheff at gcc dot gnu.org
         Resolution|                            |FIXED
   Target Milestone|---                         |4.7.0

--- Comment #5 from Mark Heffernan <meheff at gcc dot gnu.org> 2011-08-23 19:27:18 UTC ---
With patch, warning is controllable with -Wfree-nonheap-object.

[Bug middle-end/38509] Bogus "attempt to free a non-heap object" warning

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2009-07-25 23:32 -------
This case needs the compiler needs to know that TestCase::TestCase():: non_heap
cannot change value.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38509

[Bug middle-end/38509] Bogus "attempt to free a non-heap object" warning

[rubidium at openttd dot org]

------- Comment #2 from rubidium at openttd dot org  2009-07-25 23:28 -------
Is there a way to disable this warning?

I've searched the manual and gcc -v --help for 'heap' and 'free'. Neither
keyword showed any promising compiler flags.

I've also looked in the code, but I cannot find a variable (like
warn_parentheses) that enables/disables the warning.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38509

[Bug middle-end/38509] Bogus "attempt to free a non-heap object" warning

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2008-12-12 22:42 -------
  static int non_heap = 0;

<bb 2>:
  if (non_heap != 0)
    goto <bb 3>;
  else
    goto <bb 4>;

<bb 3>:
  __builtin_free (&non_heap);

Hmm, since the address of non_heap is taken, we don't optimize away the
conditional.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|c++                         |middle-end
           Keywords|                            |diagnostic, missed-
                   |                            |optimization


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38509