bugTiny C Compiler - Bugs: bug #42891, --whole-archive support

 
 

bug #42891: --whole-archive support

Submitter:  Reuben Thomas <rrt>
Submitted:  Thu 31 Jul 2014 03:53:14 PM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 03 May 2016 10:54:11 AM UTC, comment #2: 

Thanks for working on this.

I tried building the project I was experimenting with (FontForge), but tcc segfaults while compiling one of the files.

However, this is a large, complex project. I suspect it would be better to test it on something smaller first.

I'm sorry that I don't have either the expertise (I don't know much about linkers) or time to investigate further at present.

I did also try building a smaller project (psutils), and that worked fine, but I don't think it needs --whole-archive support. I am not sure what a minimal test would be.

Reuben Thomas <rrt>
Tue 03 May 2016 10:23:20 AM UTC, comment #1: 
Sergey Korshunoff <seyko>
Thu 31 Jul 2014 03:53:14 PM UTC, original submission:  

I resurrected the patch supplied to the mailing list in 2009:

https://www.mail-archive.com/tinycc-devel@nongnu.org/msg02367.html

I fixed it up to build with git head, built it and tested it. It runs fine to build binaries with an autotools build system configured with

CC=tcc ./configure

but the resultant binary comes up with an "undefined symbol" error in one of the libraries in the project. I also had to add support for -rdynamic to libtool to get the build to work, so it's possible the problem lies here and not in --whole-archive. In particular, the undefined symbol in question is not from an object that was linked with --whole-archive.

Hence, since --whole-archive is a useful flag to get tcc working with autotools, and of course in its own right, I suggest you have a look at the patch and see if it is acceptable. I cannot see any suggestion that it was actively rejected last time round, just no evidence that it was ever added.

Reuben Thomas <rrt>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by seyko (Posted a comment)
  • -email is unavailable- added by rrt (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-07-31 rrt Attached File- Added 0001-Add-no-whole-archive-support.patch, #31810

    Back to the top

    Powered by Savane 3.13-8ccc.
    Corresponding source code