Opened 9 years ago

Closed 9 years ago

#4854 closed bug (fixed)

Validating on a PPC Mac OS X: Fix miscellaneous errors and warnings

Reported by: thorkilnaur Owned by: igloo
Priority: normal Milestone: 7.2.1
Component: Compiler Version: 7.1
Keywords: Cc:
Operating System: MacOS X Architecture: powerpc
Type of failure: Building GHC failed Test Case:
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

Description

Validating a recent GHC HEAD with

$ uname -a
Darwin thorkil-naurs-mac-mini.local 9.8.0 Darwin Kernel Version 9.8.0: Wed Jul 15 16:57:01 PDT 2009; root:xnu-1228.15.4~1/RELEASE_PPC Power Macintosh
$ ghc --version
The Glorious Glasgow Haskell Compilation System, version 7.0.1
$

produces:

$ "/Users/thorkilnaur/tn/install/ghc-7.0.1/bin/ghc"   -H32m -O -Wall -Werror -H64m -O0  -package-conf libraries/bootstrapping.conf  -package-name ghc-7.1 -hide-all-packages -i -icompiler/basicTypes -icompiler/cmm -icompiler/codeGen -icompiler/coreSyn -icompiler/deSugar -icompiler/ghci -icompiler/hsSyn -icompiler/iface -icompiler/llvmGen -icompiler/main -icompiler/nativeGen -icompiler/parser -icompiler/prelude -icompiler/profiling -icompiler/rename -icompiler/simplCore -icompiler/simplStg -icompiler/specialise -icompiler/stgSyn -icompiler/stranal -icompiler/typecheck -icompiler/types -icompiler/utils -icompiler/vectorise -icompiler/stage1/build -icompiler/stage1/build/autogen -Icompiler/stage1/build -Icompiler/stage1/build/autogen -Icompiler/stage1 -Icompiler/../libraries/base/cbits -Icompiler/../libraries/base/include -Icompiler/. -Icompiler/parser -Icompiler/utils    -optP-include -optPcompiler/stage1/build/autogen/cabal_macros.h -package Cabal-1.11.0 -package array-0.3.0.2 -package base-4.3.0.0 -package bin-package-db-0.0.0.0 -package bytestring-0.9.1.8 -package containers-0.4.0.0 -package directory-1.1.0.0 -package filepath-1.2.0.0 -package hpc-0.5.0.6 -package old-time-1.0.0.6 -package process-1.0.1.4 -package unix-2.4.1.0  -Wall -fno-warn-name-shadowing -fno-warn-orphans -XCPP -XMagicHash -XUnboxedTuples -XPatternGuards -XForeignFunctionInterface -XEmptyDataDecls -XTypeSynonymInstances -XMultiParamTypeClasses -XFlexibleInstances -XRank2Types -XScopedTypeVariables -XDeriveDataTypeable -DSTAGE=1 -no-user-package-conf -rtsopts  -O     -odir compiler/stage1/build -hidir compiler/stage1/build -stubdir compiler/stage1/build -hisuf hi -osuf  o -hcsuf hc -c compiler/nativeGen/AsmCodeGen.lhs -o compiler/stage1/build/AsmCodeGen.o

compiler/nativeGen/AsmCodeGen.lhs:845:47:
    Not in scope: `rtsPackageId'

compiler/nativeGen/AsmCodeGen.lhs:849:47:
    Not in scope: `rtsPackageId'

compiler/nativeGen/AsmCodeGen.lhs:853:47:
    Not in scope: `rtsPackageId'
$

Attached a patch that seems to fix his problem. Then:

compiler/nativeGen/AsmCodeGen.lhs:594:17:
    Warning: Defined but not used: `addr'

compiler/nativeGen/AsmCodeGen.lhs:601:17:
    Warning: Defined but not used: `addr'

<no location info>:
Failing due to -Werror.

A second attached patch fixes this. Third:

compiler/nativeGen/AsmCodeGen.lhs:76:1:
    Warning: The import of `opt_Static'
             from module `StaticFlags' is redundant

<no location info>:
Failing due to -Werror.

Fixed by the third patch attached. Fourth:

compiler/vectorise/Vectorise/Utils/PRDict.hs:16:1:
    Warning: The import of `TyCon' is redundant
               except perhaps to import instances from `TyCon'
             To import instances alone, use: import TyCon()

compiler/vectorise/Vectorise/Utils/PRDict.hs:17:1:
    Warning: The import of `Outputable' is redundant
               except perhaps to import instances from `Outputable'
             To import instances alone, use: import Outputable()

<no location info>:
Failing due to -Werror.

Fixed by a fourth patch. Then:

compiler/vectorise/Vectorise/Utils.hs:7:3:
    Warning: `prDFunOfTyCon' is exported by `module Vectorise.Utils.PRDict' and `module Vectorise.Utils.Base'

<no location info>:
Failing due to -Werror.

Which a fifth patch fixes. And:

"inplace/bin/ghc-stage1" -optc-Wall -optc-Werror -optc-Wall -optc-Wextra -optc-Wstrict-prototypes -optc-Wmissing-prototypes -optc-Wmissing-declarations -optc-Winline -optc-Waggregate-return -optc-Wpointer-arith -optc-Wmissing-noreturn -optc-Wnested-externs -optc-Wredundant-decls -optc-Iincludes -optc-Irts -optc-DCOMPILING_RTS -optc-DUSE_LIBFFI_FOR_ADJUSTORS -optc-fno-strict-aliasing -optc-fno-common -optc-Ilibffi/build/include -optc-DDTRACE -optc-fomit-frame-pointer -optc-DRtsWay=\"rts_v\"   -H32m -O -Wall -Werror -H64m -O0 -Iincludes -Irts -DCOMPILING_RTS -package-name rts  -dcmm-lint  -Ilibffi/build/include -DDTRACE     -i -irts -irts/dist/build -irts/dist/build/autogen -Irts/dist/build -Irts/dist/build/autogen          -optc-O2   -c rts/Threads.c -o rts/dist/build/Threads.o
cc1: warnings being treated as errors
rts/Threads.c: In function 'threadStackUnderflow':

rts/Threads.c:670:0:
     warning: comparison between signed and unsigned

(This was also reported in http://www.haskell.org/pipermail/cvs-ghc/2010-December/058433.html.) With a sixth patch, we get past this obstacle as well.

Best regards Thorkil

Change History (12)

Changed 9 years ago by thorkilnaur

comment:1 Changed 9 years ago by thorkilnaur

And one attachment was included twice by a mistake.

Best regards Thorkil

comment:2 Changed 9 years ago by igloo

Status: newpatch

Thanks for the patches! We'll take a look.

comment:3 Changed 9 years ago by igloo

Milestone: 7.0.3

comment:4 Changed 9 years ago by igloo

Owner: set to igloo

comment:5 Changed 9 years ago by igloo

Resolution: fixed
Status: patchclosed

Thanks; all applied or already fixed.

Note: See TracTickets for help on using tickets.