Re: Certainly not a typo in binutils-2.21.51/bfd/configure.com for VMS/Alpha targets !

[Tristan Gingold <gingold at adacore dot com>]

Ok, I really have to update these DCL commands...

Tristan.

On Mar 30, 2011, at 2:17 PM, Philippe Vouters wrote:

> 
> 
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Dear binutils maintainers,
> 
> Surely an outdated information in binutils-2.21.51/bfd/configure.com
> for VMS/Alpha targets !
> 
> $ set def PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51]
> $ @setup all
> Configuring BFD for alpha target
> Generate `bfdver.h' from 'version.h' and `configure.in'.
> Generate `targmatch.h'
> Generate `config.h'
> Copy sysdep.h
> Generate build.com
> Generate libiberty build.com
> Generate opcodes/build.com
> Configuring binutils for alpha target
> Created `config.h'
> Generate binutils build.com
> Generate targ-cpu.[ch]
> Generate targ-env.h
> Generate obj-format.[ch]
> Generate atof-targ.c
> Generate gas/config.h
> Generate gas/build.com
> Compiling cpu-alpha.c
> Compiling vms.c
> %CC-F-OPENIN, error opening
> PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]vms.c
> ; as input
> - -RMS-E-FNF, file not found
> $ dire PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]vms*
> 
> Directory PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]
> 
> vms-alpha.c;1       vms-lib.c;1         vms-misc.c;1       
> vms.h;1           
> 
> Total of 4 files.
> 
> Also lacking vms-hdr.c, vms-gsd.c, vms-tir.c source files !
> 
> vms-alpha.c lacking :
> #ifdef __DECC
> #include <inttypes.h>
> #endif
> This is necessary to correctly define intptr_t used in the code.
> 
> In the very same source code:
> Compiling vms-alpha.c
> 
>  if (PRIV (recrd.rec_size) <= 0)
> ......^
> %CC-I-QUESTCOMPARE, In this statement, the unsigned expression
> "(((struct vms_pr
> ivate_data_struct ...)(abfd)->tdata.any)->recrd.rec_size)" is being
> compared with a relational operator to a constant whose value is not
> greater than zero.  This might not be
> what you intended.
> at line number 806 in file
> PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]vms-alpha.c;2
> 
>          if (section->reloc_count <= 0)
> ..............^
> %CC-I-QUESTCOMPARE, In this statement, the unsigned expression
> "section->reloc_count" is being compared with a relational operator to
> a constant whose value is not greater than zero.  This might not be
> what you intended.
> at line number 3757 in file
> PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]vms-alpha.c;2
> 
> Proposed correction:
> if ((int)PRIV (recrd.rec_size) <= 0) at line 806
> if ((int)section->reloc_count <= 0) at line 3757
> 
> Compiling targets.c
> 
>        &DEFAULT_VECTOR,
> .........^
> %CC-E-UNDECLARED, In the initializer for bfd_default_vector[0],
> "bfd_alpha_vec"
> is not declared.
> at line number 1316 in file
> PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.bfd]targe
> ts.c;1
> 
> The above error is caused by my previous changes to bfd/configure.com.
> 
> Proposed changes to bfd/configure.com in the Alpha section:
> $ DEFS="""HAVE_vms_alpha_vec"","+-
>    """DEFAULT_VECTOR=vms_alpha_vec"","+-
>    """SELECT_VECS=&vms_alpha_vec"","+-
>    """SELECT_ARCHITECTURES=&bfd_alpha_arch"""
> $ FILES="cpu-alpha,vms-alpha,vms-lib,vms-misc,"
> 
> With all these Alpha/VMS specific changes after my changes for IA64/VMS, a
>                    $ sho sys/noproc
>                    OpenVMS V8.3  on node LION  30-MAR-2011
> 08:00:00.34  Uptime  86 01:30:18
>                    $ sho cpu
> 
>                    System: LION, AlphaServer ES40
> 
>                    CPU ownership sets:
>                   Active               0-3
>                   Configure            0-3
> 
>                    CPU state sets:
>                   Potential            0-3
>                   Autostart            0-3
>                   Powered Down         None
>                   Not Present          None
>                   Hard Excluded        None
>                   Failover             None
>                    $ show default
>                      PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51]
>                    $ @setup all
> cleanly builds everything with absolutely no
> informational/warning/error message.
> 
> Quick check:
> 
> $ nm :== $PTAC$DKA0:[PVOUTERS.PHV.binutils-2^.21^.51.binutils]nm.EXE
> $ pipe nm sys$share:DECC$SHR.EXE | search sys$pipe: gmtime
> 0023e650 A DECC$GMTIME
> 0023e640 A DECC$GMTIME_R
> 00260550 A DECC$__UTCTZ_GMTIME
> 002604b8 A DECC$__UTCTZ_GMTIME_R
> 0023a410 A DECC$__UTC_GMTIME
> 0023a378 A DECC$__UTC_GMTIME_R
> 0023a410 A decc$__utc_gmtime
> 0023a378 A decc$__utc_gmtime_r
> 00260550 A decc$__utctz_gmtime
> 002604b8 A decc$__utctz_gmtime_r
> 0023e650 A decc$gmtime
> 0023e640 A decc$gmtime_r
> 
> Yours truly,
> Philippe
> 
> - -- 
> Philippe Vouters (Fontainebleau/France)
> URL: http://vouters.dyndns.org/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.11 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
> 
> iEYEARECAAYFAk2TH00ACgkQsUYBDUbOpdpqiQCfbRnEuJLZzr8v4pS+jjokNumY
> yJYAnjfm8Ir7zBoYs0VGjMpNk7+Ab3iU
> =9yyA
> -----END PGP SIGNATURE-----
> 
>