Re: -.su file in kernel compile dir

2020-07-07 Thread Christos Zoulas
In article <20200707044652.c094d4e...@thoreau.thistledown.com.au>,
Simon Burge   wrote:
>Valery Ushakov wrote:
>
>> Recent changes to record stack usage cause a file named -.su to be
>> created (that refers assym.c).  It plays tricks with targets like
>> clean that refer to *.su
>
>This -.su file is created by genassym which calls GCC using stdin for
>the input file.  The attached patch feels a bit hackish, but works for
>me.  Any reason not to commit it now, or does anyone have a cleaner
>fix?  I certainly didn't want to add all the ugliness of checking if the
>current compiler is GCC and !vax !

Just add another -N-fstack-usage?

-   ${GENASSYM} -- ${CC} ${CFLAGS:N-Wa,*} ${CPPFLAGS} ${PROF} \
+   ${GENASSYM} -- ${CC} ${CFLAGS:N-Wa,*:N-fstack-usage} ${CPPFLAGS} \
+   ${PROF} \

christos



Re: -.su file in kernel compile dir

2020-07-06 Thread Martin Husemann
On Tue, Jul 07, 2020 at 02:46:52PM +1000, Simon Burge wrote:
>   ${GENASSYM} -- ${CC} ${CFLAGS:N-Wa,*} ${CPPFLAGS} ${PROF} \
>   ${GENASSYM_CPPFLAGS} > assym.h.tmp && \

Can we instead delete the stack usage option in CFLAGS, CPPFLAGS or
GENASSYM_CPPFLAGS (wherever it ends up now)?

Martin


Re: -.su file in kernel compile dir

2020-07-06 Thread Simon Burge
Valery Ushakov wrote:

> Recent changes to record stack usage cause a file named -.su to be
> created (that refers assym.c).  It plays tricks with targets like
> clean that refer to *.su

This -.su file is created by genassym which calls GCC using stdin for
the input file.  The attached patch feels a bit hackish, but works for
me.  Any reason not to commit it now, or does anyone have a cleaner
fix?  I certainly didn't want to add all the ugliness of checking if the
current compiler is GCC and !vax !

Cheers,
Simon.
--
Index: assym.mk
===
RCS file: /cvsroot/src/sys/conf/assym.mk,v
retrieving revision 1.5
diff -d -p -u -r1.5 assym.mk
--- assym.mk10 Sep 2015 13:11:39 -  1.5
+++ assym.mk7 Jul 2020 04:43:15 -
@@ -6,6 +6,7 @@ assym.h: ${GENASSYM_CONF} ${GENASSYM_EXT
${GENASSYM} -- ${CC} ${CFLAGS:N-Wa,*} ${CPPFLAGS} ${PROF} \
${GENASSYM_CPPFLAGS} > assym.h.tmp && \
mv -f assym.h.tmp assym.h
+   rm -f ./-.su
 
 .if !defined(___USE_SUFFIX_RULES___)
 ${SRCS:T:M*.[sS]:C|\.[Ss]|.o|}: assym.h


-.su file in kernel compile dir

2020-07-05 Thread Valery Ushakov
Recent changes to record stack usage cause a file named -.su to be
created (that refers assym.c).  It plays tricks with targets like
clean that refer to *.su

-uwe