Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 343886

Summary: remove .clink and __ED from big.c for all non-whole_program profiles
Product: [Technology] RTSC Reporter: Karl Wechsler <karl>
Component: CoreAssignee: Sasha Slijepcevic <sascha>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: dfriedland
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:

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