* ELLCC updated to musl 0.9.14
@ 2013-09-29 13:24 Richard Pennington
2013-09-29 17:26 ` Rich Felker
0 siblings, 1 reply; 2+ messages in thread
From: Richard Pennington @ 2013-09-29 13:24 UTC (permalink / raw)
To: musl
Hi,
I've updated the musl version of the clang/LLVM based ELLCC compiler
suite to use musl release 0.9.14.
As usual the musl guys have been doing a great job. The new release
looks great!
http://ellcc.org
I dis have to make a few small changes to the configure script. The main
one was that the warning the clang gives if a warning option is not
supported is a warning, not an error. I added -Werror to tryflag. Other
than that I just loosened up the target names a bit.
-- /home/rich/configure 2013-09-28 09:40:01.387766267 -0500
+++ configure 2013-09-28 09:52:55.758633462 -0500
@@ -79,7 +79,7 @@
tryflag () {
printf "checking whether compiler accepts %s... " "$2"
echo "typedef int x;" > "$tmpc"
-if $CC "$2" -c -o /dev/null "$tmpc" >/dev/null 2>&1 ; then
+if $CC -Werror "$2" -c -o /dev/null "$tmpc" >/dev/null 2>&1 ; then
printf "yes\n"
eval "$1=\"\${$1} \$2\""
eval "$1=\${$1# }"
@@ -225,9 +225,10 @@
arm*) ARCH=arm ;;
i?86*) ARCH=i386 ;;
x86_64*) ARCH=x86_64 ;;
-mips-*|mipsel-*) ARCH=mips ;;
-microblaze-*) ARCH=microblaze ;;
+mips*|mipsel*) ARCH=mips ;;
+microblaze*) ARCH=microblaze ;;
powerpc-*) ARCH=powerpc ;;
+ppc*) ARCH=powerpc ;;
unknown) fail "$0: unable to detect target arch; try $0 --target=..." ;;
*) fail "$0: unknown or unsupported target \"$target\"" ;;
esac
-Rich
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: ELLCC updated to musl 0.9.14
2013-09-29 13:24 ELLCC updated to musl 0.9.14 Richard Pennington
@ 2013-09-29 17:26 ` Rich Felker
0 siblings, 0 replies; 2+ messages in thread
From: Rich Felker @ 2013-09-29 17:26 UTC (permalink / raw)
To: musl
On Sun, Sep 29, 2013 at 08:24:27AM -0500, Richard Pennington wrote:
> Hi,
>
> I've updated the musl version of the clang/LLVM based ELLCC compiler
> suite to use musl release 0.9.14.
> As usual the musl guys have been doing a great job. The new release
> looks great!
>
> http://ellcc.org
Nice to hear!
> I dis have to make a few small changes to the configure script. The
> main one was that the warning the clang gives if a warning option is
> not supported is a warning, not an error. I added -Werror to
> tryflag. Other than that I just loosened up the target names a bit.
>
> -- /home/rich/configure 2013-09-28 09:40:01.387766267 -0500
> +++ configure 2013-09-28 09:52:55.758633462 -0500
> @@ -79,7 +79,7 @@
> tryflag () {
> printf "checking whether compiler accepts %s... " "$2"
> echo "typedef int x;" > "$tmpc"
> -if $CC "$2" -c -o /dev/null "$tmpc" >/dev/null 2>&1 ; then
> +if $CC -Werror "$2" -c -o /dev/null "$tmpc" >/dev/null 2>&1 ; then
My concern here is that gratuitous warnings for unrelated reasons
(e.g. a weird but valid flag added into $CC) could prevent detection
of supported flags. Here, false negatives (failing to detect support
for a flag that's actually needed) are worse than false positives
(detecting support for a flag that will spam warnings during build).
I'd rather find some other method of solving this problem with clang
and other compilers that are treating unsupported gcc options as
warnings rather than errors...
> -mips-*|mipsel-*) ARCH=mips ;;
> -microblaze-*) ARCH=microblaze ;;
> +mips*|mipsel*) ARCH=mips ;;
> +microblaze*) ARCH=microblaze ;;
For microblaze this is probably correct (to also get microblazeel or
whatever), but for mips, it will wrongly pick up mips64 as being mips.
When mips64 support is added, it will be a separate arch and thus need
a separate case here.
> powerpc-*) ARCH=powerpc ;;
> +ppc*) ARCH=powerpc ;;
This could be simplified as:
powerpc-*|ppc-*) ARCH=powerpc ;;
Rich
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-09-29 17:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-09-29 13:24 ELLCC updated to musl 0.9.14 Richard Pennington
2013-09-29 17:26 ` Rich Felker
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/musl/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).