WITH_DEBUG too restrictive on CFLAGS?

[Sean McNeil]

I have found recently that the following ports will not compile on amd64
when WITH_DEBUG is set to "YES":

	converters/recode
	devel/pwlib

searching for all ports that set CFLAGS with -fPIC, I find:

	archivers/p7zip
	archivers/rpm4
	archivers/grzip
	audio/anthem
	audio/bmp-wma
	audio/liba52
	audio/libworkman
	audio/osalp
	audio/swhplugins
	audio/xmms-a52dec
	...

You get the idea - it is used pervasively.

The problem is, when WITH_DEBUG is YES, /usr/ports/Mk/bsd.port.mk will
strip out all flags from CFLAGS of the type -O* and -f*.  I don't think
this is a good idea.  First, the debugger can handle optimized code
pretty well and I can't build ports with debug introduced yet have ports
run optimized.  Second, -f* is a little over-aggressive when the one
flag that really must be removed is -fomit-frame-pointer.

Would it be possible to have this reviewed and, hopefully, changed to
allow -O and -f switches and just eliminate -fomit-frame-pointer?

Cheers,
Sean