bug in ipf/tools/ipfcomp.c

[Matthias Drochner]

Hi -
I'm not using ipf myself, but clang has found an obvious bug
which might lead to an endless loop.
I think the appended patch fixes it, but I'll leave it to
someone who is familiar with the code to decide.
best regards
Matthias



------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------
Forschungszentrum Juelich GmbH
52425 Juelich
Sitz der Gesellschaft: Juelich
Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
Vorsitzende des Aufsichtsrats: MinDir'in Baerbel Brumme-Bothe
Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
Dr. Ulrich Krafft (stellv. Vorsitzender), Prof. Dr.-Ing. Harald Bolt,
Prof. Dr. Sebastian M. Schmidt
------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------

[Thor Lancelot Simon]

Post by Matthias Drochner
Hi -
I'm not using ipf myself, but clang has found an obvious bug
which might lead to an endless loop.

You're building NetBSD with clang?

Thor

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de

[Matthias Drochner]

Post by Thor Lancelot Simon
You're building NetBSD with clang?

Yes, but not as a fully integrated system build yet because
it needs hacks to share/mk/* to deal with the different
warning flags... most notably that clang understands a
"-Wunknown-warning-option" to not complain about gcc specific
options but gcc complains about exactly that.
And there are problems with asm code because clang doesn't
support traditional-cpp.
But besides of this, kernels compiled with clang run well,
and I've built big parts of userland (which is
bin/sbin/usr.bin/usr.sbin except cpuctl and ndiscvt).

best regards
Matthias



------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------
Forschungszentrum Juelich GmbH
52425 Juelich
Sitz der Gesellschaft: Juelich
Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
Vorsitzende des Aufsichtsrats: MinDir'in Baerbel Brumme-Bothe
Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
Dr. Ulrich Krafft (stellv. Vorsitzender), Prof. Dr.-Ing. Harald Bolt,
Prof. Dr. Sebastian M. Schmidt
------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de

[Jeff Rizzo]

Post by Matthias Drochner

Post by Thor Lancelot Simon
You're building NetBSD with clang?

Yes, but not as a fully integrated system build yet because
it needs hacks to share/mk/* to deal with the different
warning flags... most notably that clang understands a

Off-topic, I know - but can I request that you send more info about what you've done to a list where it *is* on-topic - I actually started looking at clang myself just this weekend, but I'm critically short of free time, and I have a devil of a time just getting it built! :)

+j


--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de

[Matthias Drochner]

[building NetBSD with clang]

Post by Jeff Rizzo
can I request that you send more info about what you've done

Sure... I've added lang/clang to pkgsrc, this should make
building the compiler easy. The rest is basically
"make CC=/usr/pkg/bin/clang".
For the NetBSD source tree changes needed -- this is OT
for tech-net indeed; I'll post a list of problems I've
found to tech-toolchain or so.

best regards
Matthias



------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------
Forschungszentrum Juelich GmbH
52425 Juelich
Sitz der Gesellschaft: Juelich
Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
Vorsitzende des Aufsichtsrats: MinDir'in Baerbel Brumme-Bothe
Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
Dr. Ulrich Krafft (stellv. Vorsitzender), Prof. Dr.-Ing. Harald Bolt,
Prof. Dr. Sebastian M. Schmidt
------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de

[Matthias Drochner]

built big parts of userland

Need to add that the "extern inline" declarations used
in signal.h and some external code cause problems too.
gcc's behaviour is non-standard here.

best regards
Matthias



------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------
Forschungszentrum Juelich GmbH
52425 Juelich
Sitz der Gesellschaft: Juelich
Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
Vorsitzende des Aufsichtsrats: MinDir'in Baerbel Brumme-Bothe
Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
Dr. Ulrich Krafft (stellv. Vorsitzender), Prof. Dr.-Ing. Harald Bolt,
Prof. Dr. Sebastian M. Schmidt
------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de