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

Issue 4886041: [lto] Rename functions in tree/data streamer (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 8 months ago by Diego Novillo
Modified:
12 years, 8 months ago
Reviewers:
CC:
rguenther_suse.de, jh_suse.cz, matz_suse.de, gcc-patches_gcc.gnu.org
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+771 lines, -824 lines) Patch
M gcc/Makefile.in View 2 chunks +4 lines, -2 lines 0 comments Download
M gcc/cp/pph-streamer.h View 10 chunks +16 lines, -15 lines 0 comments Download
M gcc/cp/pph-streamer-in.c View 4 chunks +8 lines, -8 lines 0 comments Download
M gcc/cp/pph-streamer-out.c View 5 chunks +8 lines, -8 lines 0 comments Download
M gcc/data-streamer.h View 10 chunks +63 lines, -56 lines 0 comments Download
M gcc/data-streamer-in.c View 6 chunks +12 lines, -33 lines 0 comments Download
M gcc/data-streamer-out.c View 6 chunks +34 lines, -54 lines 0 comments Download
M gcc/gimple-streamer-in.c View 7 chunks +21 lines, -21 lines 0 comments Download
M gcc/gimple-streamer-out.c View 9 chunks +29 lines, -28 lines 0 comments Download
M gcc/ipa-inline-analysis.c View 9 chunks +36 lines, -45 lines 0 comments Download
M gcc/ipa-prop.c View 16 chunks +30 lines, -36 lines 0 comments Download
M gcc/ipa-pure-const.c View 6 chunks +6 lines, -6 lines 0 comments Download
M gcc/ipa-reference.c View 10 chunks +14 lines, -13 lines 0 comments Download
M gcc/lto-cgraph.c View 39 chunks +105 lines, -104 lines 0 comments Download
M gcc/lto-section-out.c View 2 chunks +2 lines, -1 line 0 comments Download
M gcc/lto-streamer.h View 4 chunks +2 lines, -12 lines 0 comments Download
M gcc/lto-streamer.c View 1 chunk +1 line, -1 line 0 comments Download
M gcc/lto-streamer-in.c View 35 chunks +65 lines, -65 lines 0 comments Download
M gcc/lto-streamer-out.c View 35 chunks +102 lines, -95 lines 0 comments Download
M gcc/lto/lto.c View 7 chunks +8 lines, -8 lines 0 comments Download
M gcc/streamer-hooks.h View 1 chunk +0 lines, -2 lines 0 comments Download
M gcc/tree-streamer.h View 3 chunks +28 lines, -29 lines 0 comments Download
M gcc/tree-streamer.c View 15 chunks +37 lines, -37 lines 0 comments Download
M gcc/tree-streamer-in.c View 26 chunks +39 lines, -39 lines 0 comments Download
M gcc/tree-streamer-out.c View 32 chunks +101 lines, -106 lines 0 comments Download

Messages

Total messages: 7
Diego Novillo
This is mind numbing but mechanical. This rename does two things: 1- Replaces the 'lto_' ...
12 years, 8 months ago (2011-08-12 13:40:18 UTC) #1
rguenther_suse.de
On Fri, 12 Aug 2011, Diego Novillo wrote: > > This is mind numbing but ...
12 years, 8 months ago (2011-08-12 13:45:43 UTC) #2
Diego Novillo
On Fri, Aug 12, 2011 at 09:45, Richard Guenther <rguenther@suse.de> wrote: > On Fri, 12 ...
12 years, 8 months ago (2011-08-12 13:47:41 UTC) #3
rguenther_suse.de
On Fri, 12 Aug 2011, Diego Novillo wrote: > On Fri, Aug 12, 2011 at ...
12 years, 8 months ago (2011-08-12 13:50:12 UTC) #4
Diego Novillo
On Fri, Aug 12, 2011 at 09:50, Richard Guenther <rguenther@suse.de> wrote: > Oh, how even ...
12 years, 8 months ago (2011-08-12 13:53:34 UTC) #5
matz_suse.de
Hi, On Fri, 12 Aug 2011, Diego Novillo wrote: > 3- Replaces dwarf terminology for ...
12 years, 8 months ago (2011-08-12 15:16:18 UTC) #6
Diego Novillo
12 years, 8 months ago (2011-08-12 15:18:21 UTC) #7
On Fri, Aug 12, 2011 at 11:16, Michael Matz <matz@suse.de> wrote:

> I see downthread that this is changed to hwi/uhwi already.  I even would
> have suggested int/uint directly, as we never stream other numbers than
> host_wide_ints or chars.

For now, that's true.  Over in PPH we do stream out ints/uints, and
those will likely move up to data-streamer.[ch].


Diego.
Sign in to reply to this message.

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