[PATCH] ld: Add '--defined' command line option.

Nick Clifton nickc@redhat.com
Thu Jul 23 11:26:00 GMT 2015


Hi Andrew,

   I like this patch in principle, but there are a few problems with it 
at the moment:

   * Is this patch really necessary ?  Can the same affect not be 
achieved by combining --undefined= and --no-undefined ?  If not, why not ?

   * I think that the name of the option, --defined is slightly 
confusing.  When I first read your submission I thought that it meant 
that a defined symbol was being created, ie that it was an alias of 
--defsym.  So how about a different name ?  For example 
--no-allow-undef=<foo>.  [This version might even be extended so the the 
inverse option --allow-undef=<foo> would stop --no-undefined from 
complaining if that particular symbol was undefined].  Or how about 
--undefined-and-err= so that the connection with --undefined is more 
obvious ?

   * The documentation needs to be extended.  You need an entry in the 
ld/NEWS file and an update to the --undefined entry in ld/ld.texinfo 
referring the reader to the new option and explaining why they might 
want to use it instead.

   * The new tests need to be extend to make sure that the linker does 
not complain if the defined symbol does exist, and that the new option 
does duplicate the behaviour of the --undefined option.

Cheers
   Nick



More information about the Binutils mailing list