Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 343886 - remove .clink and __ED from big.c for all non-whole_program profiles
Summary: remove .clink and __ED from big.c for all non-whole_program profiles
Status: CLOSED FIXED
Alias: None
Product: RTSC
Classification: Technology
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Sasha Slijepcevic CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-26 13:29 EDT by Karl Wechsler CLA
Modified: 2015-02-14 22:39 EST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Karl Wechsler CLA 2011-04-26 13:29:34 EDT 
We need to make big.c smaller and remove the wpod workarounds for the profiles that are not using whole_program.

we attempted to remove .clink but we are still having problems building the stairstep example. 

need to revisit this and do both when we have time to adequately test.
Comment 1 Sasha Slijepcevic CLA 2011-05-12 20:52:23 EDT 
My tests show that .clink is always needed for COFF, no matter which profile is selected, and .clink is never needed for ELF, no matter which profile is selected.

If .clink directives are not used in cases for which I wrote they are "needed", the code sizie increases for about 20% for the stairstep example on 64P.

#pragma DATA_SECTION is always needed.
Comment 2 Sasha Slijepcevic CLA 2011-05-19 15:15:23 EDT 
.clink part was fixed as described in 345294. This bug will stay open for the __ED problem.
Comment 3 Sasha Slijepcevic CLA 2012-01-10 15:39:50 EST 
_ED constants are removed from non-wholeprogram builds in 3.23.
Comment 4 Karl Wechsler CLA 2012-02-08 14:45:18 EST 
Looks like actually fixed in 3.22.02 ...

--------

For simple hello/66/ELF app …

File went from 850,000 characters to 680,000 characters  20%!

File size from 18,507 -> 16,891 lines -> 10%.

Thanks,
-Karl-

________________________________________
From: Slijepcevic, Sasa 
Sent: Wednesday, February 08, 2012 10:59 AM
To: Wechsler, Karl
Subject: RE: when did you fix the __ED problem?

I think you’ll have to go back to 3.22.02 to see _ED for non -whole_program builds.
Comment 5 Dave Russo CLA 2015-02-14 22:39:44 EST 
clean out old verified bugs