incubator-stdcxx-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Black <abl...@roguewave.com>
Subject [PATCH]Assorted cleanup
Date Wed, 08 Nov 2006 18:24:06 GMT
Greetings all.

Attached is a patch that aims to resolve a couple minor compile issues 
with the exec utility.

The first concern is the current link behavior, where the standard 
library is linked into all executables.  While this is normally 
desirable, it is considered undesirable for the exec utility, which is 
designed without the use of the standard library.  The tactic I chose to 
use in this patch was to filter the library out of the $LDFLAGS 
variable, but I wonder if a better approach would be alter the 
makefile.common file so that the library isn't included by default on 
the link line.

The second concern is a compile failure when using the Compaq compiler. 
  In particular, this compiler fails on code that was implemented as a 
workaround for STDCXX-291.  I feel that it is (slightly) more efficient 
to use the older code than the STDCXX-291 workaround code, so I chose to 
make that the default path, but it would also be possible to make the 
alternate code path specific to the Compaq compiler (via the __DECCXX macro)

--Andrew Black

Changelog:
	* GNUmakefile.bin (LDFLAGS.exec): Define LDFLAGS variant, filtering out 
the stdcxx library
	  (exec): use LDFLAGS.exec rather than LDFLAGS
	* exec.cpp (wait_for_child): Use workaround for STDCXX-291 only with HP 
aCC (causes failure with Compaq CXX)

Mime
View raw message