This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [gold] PATCH: Filter out -Wp,-D_FORTIFY_SOURCE=. with -O0


>> 2012-10-25  H.J. Lu  <hongjiu.lu@intel.com>
>>
>>         * testsuite/Makefile.am (COMPILE1): New variable. Renamed from
>>         COMPILE generated by automake.
>>         (LINK1): Likewise.
>>         (CXXCOMPILE1): Likewise.
>>         (CXXLINK1): Likewise.
>>         (COMPILE): Strip out -Wp,-D_FORTIFY_SOURCE= from COMPILE1.
>>         (LINK): Likewise.
>>         (CXXCOMPILE): Likewise.
>>         (CXXLINK): Likewise.
>>         * testsuite/Makefile.in: Regenerated.
>
> This is OK.

Wouldn't it have worked to add -U_FORTIFY_SOURCE to CPPFLAGS? It seems
like that would have been a lot cleaner. (All those `echo ... | sed
...` in my make output hurt my eyes and make it a pain to cut and
paste compile commands.)

-cary


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]