Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(3465)

Issue 7241067: [Google 4.7 Split Dwarf] Use .debug_str for some strings.

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 1 month ago by saugustine
Modified:
11 years, 1 month ago
Reviewers:
Cary
CC:
gcc-patches_gcc.gnu.org
Base URL:
svn+ssh://gcc.gnu.org/svn/gcc/branches/google/gcc-4_7/
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+107 lines, -41 lines) Patch
M gcc/dwarf2out.c View 18 chunks +107 lines, -41 lines 0 comments Download

Messages

Total messages: 5
saugustine
The enclosed patch for Google 4.7 is an optimization for debug strings under -gsplit-dwarf. Currently ...
11 years, 1 month ago (2013-02-01 23:26:45 UTC) #1
Cary
> @@ -22385,21 +22448,21 @@ > static void > output_indirect_strings (void) > { > + ...
11 years, 1 month ago (2013-02-04 19:56:53 UTC) #2
saugustine
On Mon, Feb 4, 2013 at 11:56 AM, Cary Coutant <ccoutant@google.com> wrote: >> @@ -22385,21 ...
11 years, 1 month ago (2013-02-04 21:21:59 UTC) #3
Cary
> Ahh, good catch. Fixed as attached. Looks good, thanks. OK for the google/gcc-4_7 branch. ...
11 years, 1 month ago (2013-02-04 21:45:16 UTC) #4
saugustine
11 years, 1 month ago (2013-02-04 22:05:51 UTC) #5
On Mon, Feb 4, 2013 at 1:45 PM, Cary Coutant <ccoutant@google.com> wrote:
>> Ahh, good catch. Fixed as attached.
>
> Looks good, thanks. OK for the google/gcc-4_7 branch.
>
> (And, yes, please do port this to trunk when Stage 1 reopens.)

Thanks. Committed.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b