[Llvm-bgq-discuss] Compiling apps on Cetus with clang

[Boisvert, Sebastien]

Is there an option to turn that into a warning ?


> ________________________________________
> From: Hal Finkel [hfinkel at anl.gov]
> Sent: Friday, October 10, 2014 2:18 PM
> To: Boisvert, Sebastien
> Cc: llvm-bgq-discuss at lists.alcf.anl.gov
> Subject: Re: [Llvm-bgq-discuss] Compiling apps on Cetus with clang
> ----- Original Message -----
> > From: "Sebastien Boisvert" <boisvert at anl.gov>
> > To: "Hal J. Finkel" <hfinkel at anl.gov>
> > Cc: llvm-bgq-discuss at lists.alcf.anl.gov
> > Sent: Friday, October 10, 2014 2:15:50 PM
> > Subject: RE: [Llvm-bgq-discuss] Compiling apps on Cetus with clang
> >
> > If I create a separate file for this broken prototype, will there be
> > further things
> > that need to be cared of ?
> No, as I recall, this is the only thing.
> -Hal
> >
> > > ________________________________________
> > > From: Hal Finkel [hfinkel at anl.gov]
> > > Sent: Friday, October 10, 2014 2:13 PM
> > > To: Boisvert, Sebastien
> > > Cc: llvm-bgq-discuss at lists.alcf.anl.gov
> > > Subject: Re: [Llvm-bgq-discuss] Compiling apps on Cetus with clang
> > > Hi,
> > > Unfortunately, this is a known problem (and is already part of a
> > > Clang-compatibility PMR with IBM).
> > > The problem really is not what is in compiler_support.h, but the
> > > fact that the declaration in process.h is missing __INLINE__ (if
> > > you'll notice, all of the other declarations have them). What I've
> > > done previously is to create a local copy of process.h, add the
> > > __INLINE__ and then put that file in my include path ahead of the
> > > real one.
> > > I should really create a proper system-wide work-around for this,
> > > but I've not yet.
> > > -Hal
> > > ----- Original Message -----
> > > > From: "Sebastien Boisvert" <boisvert at anl.gov>
> > > > To: llvm-bgq-discuss at lists.alcf.anl.gov
> > > > Sent: Friday, October 10, 2014 2:09:35 PM
> > > > Subject: [Llvm-bgq-discuss] Compiling apps on Cetus with clang
> > > >
> > > >
> > > >
> > > > I have set +mpiwrapper-bgclang in my ~/.soft
> > > >
> > > > Then I ran 'resoft'.
> > > >
> > > > My build script is:
> > > >
> > > > [boisvert at cetuslac1 biosal]$ cat
> > > > scripts/IBM_Blue_Gene_Q/build-spate-clang.sh
> > > > #!/bin/bash
> > > >
> > > > # on Cetus:
> > > > # 1. edit ~/.soft
> > > > # 2. run 'resoft'
> > > >
> > > > CFLAGS="-I. -g -O3 -DTHORIUM_DEBUG "
> > > > CFLAGS="$CFLAGS -I/soft/libraries/alcf/current/xl/ZLIB/include"
> > > > LDFLAGS="-L/soft/libraries/alcf/current/xl/ZLIB/lib -lm -lz"
> > > >
> > > > make clean
> > > > make -j 8 CFLAGS="$CFLAGS" LDFLAGS="$LDFLAGS" \
> > > >     applications/spate_metagenome_assembler/spate
> > > >
> > > >
> > > > I am getting this error:
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > Clang 3.6.0
> > > > "clang version 3.6.0 (trunk)"
> > > > CFLAGS="-I. -g -O3 -DTHORIUM_DEBUG "
> > > >
> > > > /bgsys/drivers/V1R2M2/ppc64/spi/include/kernel/cnk/process_impl.h:132:10:
> > > > error: static declaration of 'Kernel_GetJobID' follows non-static
> > > > declaration
> > > >
> > > >
> > > > Obviously, I am not the author of process_impl.h.
> > > >
> > > > The line 132 is:
> > > >
> > > > __INLINE__
> > > > uint64_t Kernel_GetJobID()
> > > > {
> > > >    return CNK_SPI_SYSCALL_0(GETJOBID);
> > > > }
> > > >
> > > >
> > > > The file /bgsys/drivers/V1R2M2/ppc64/spi/include/kernel/process.h
> > > > contains the prototype:
> > > >
> > > > /*! \brief Returns the JobID of the currently active job
> > > >  *
> > > >  * \return job identifier
> > > >  */
> > > > uint64_t Kernel_GetJobID();
> > > >
> > > >
> > > > I am not sure what is the problem. I suppose that it should also
> > > > be
> > > > __INLINE__  (whatever this is) in process.h.
> > > >
> > > >
> > > > I digged a little and __INLINE__ is defined in
> > > > /bgsys/drivers/V1R2M2/ppc64/hwi/include/common/compiler_support.h
> > > > at
> > > > line 28:
> > > >
> > > > #ifndef __INLINE__
> > > > #ifdef __llvm__
> > > > #define __INLINE__ static inline
> > > > #else
> > > > #define __INLINE__ extern inline __attribute__((always_inline))
> > > > #endif
> > > > #endif
> > > >
> > > >
> > > > So basically this is broken because a function can not be not
> > > > static
> > > > (in process.h) and static (process_impl.h) at the same time.
> > > >
> > > > The obvious workaround here is to fix the broken
> > > > compiler_support.h.
> > > >
> > > >
> > > >
> > > > Is there a (easy) of solving this (without having to re-root the
> > > > whole V1R2M2 tree !) ?
> > > >
> > > > _______________________________________________
> > > > llvm-bgq-discuss mailing list
> > > > llvm-bgq-discuss at lists.alcf.anl.gov
> > > > https://lists.alcf.anl.gov/mailman/listinfo/llvm-bgq-discuss
> > > >
> > > --
> > > Hal Finkel
> > > Assistant Computational Scientist
> > > Leadership Computing Facility
> > > Argonne National Laboratory
> >
> >
> --
> Hal Finkel
> Assistant Computational Scientist
> Leadership Computing Facility
> Argonne National Laboratory