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

 
 

bug #42891: --whole-archive support

Submitted by:  Reuben Thomas <rrt>
Submitted on:  Thu 31 Jul 2014 03:53:14 PM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: None
Privacy: PublicAssigned to: None
Open/Closed: Open

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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:

Please test https://github.com/seyko2/tinycc/commit/65e42df4d0fedb340f42652bb49fdacc7bc3e05c

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 File(s):
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 31 Jul 2014 03:53:14 PM UTCrrtAttached File-=>Added 0001-Add-no-whole-archive-support.patch, #31810

    Back to the top


    Powered by Savane 3.1-cleanup1