On Fri, Jan 13, 2012 at 10:17 PM, Amker.Cheng <amker.ch...@gmail.com> wrote: > On Fri, Jan 13, 2012 at 5:33 PM, Richard Guenther > <richard.guent...@gmail.com> wrote: >> >> No, I think the check is superfluous and should be removed. I also wonder >> why we exempt BUILT_IN_FREE here ... can you dig in SVN history a bit? >> For both things?
Hi Richard, The BUILT_IN_FREE was introduced in r138362 fixing PR36970, in which gcc did not give warning on freeing non-heap memory, as in program: main () { char array[100]; free (array); } I will run make check to see whether it's ok we do not check DECL_ASSEMBLER_NAME_SET_P and send a patch then. BTW, should I create a bug for this? Thanks.