Committed to google/gcc-4_6 after validation.
On 2012/03/21 05:07:33, davidxl wrote:
ok for google branches after checkin validation.
David
http://codereview.appspot.com/5851044/
ok for google branches after checkin validation.
David
http://codereview.appspot.com/5851044/
On 2012/03/20 19:26:26, eraman wrote:
http://codereview.appspot.com/5851044/diff/1/callgraph.c
File callgraph.c (right):
http://codereview.appspot.com/5851044/diff/1/callgraph.c#newcode513
callgraph.c:513: const int section_priority[] = {0, 3, 4, 2, 1};
Add a comment about section_priority
D
Uploaded new patch.
On 2012/03/20 19:25:38, davidxl wrote:
It would be nice to add some unit/regression test cases of some sort.
Made the existing unit test case check the final layout.
David
http://codereview.appspot.com/5851044/diff/1/callgraph.c
File callgraph.c (right):
http://cod
http://codereview.appspot.com/5851044/diff/1/callgraph.c
File callgraph.c (right):
http://codereview.appspot.com/5851044/diff/1/callgraph.c#newcode513
callgraph.c:513: const int section_priority[] = {0, 3, 4, 2, 1};
Add a comment about section_priority
http://codereview.appspot.com/5851044/diff
It would be nice to add some unit/regression test cases of some sort.
David
http://codereview.appspot.com/5851044/diff/1/callgraph.c
File callgraph.c (right):
http://codereview.appspot.com/5851044/diff/1/callgraph.c#newcode309
callgraph.c:309: if (!is_prefix_of ("_ZL", name))
How about static