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

Issue 5620047: [pph] Minimal type merging. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 1 month ago by Lawrence Crowl
Modified:
11 years, 10 months ago
Reviewers:
Diego Novillo
CC:
gcc-patches_gcc.gnu.org
Base URL:
svn+ssh://gcc.gnu.org/svn/gcc/branches/pph/
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+144 lines, -31 lines) Patch
M gcc/cp/error.c View 2 chunks +7 lines, -2 lines 0 comments Download
M gcc/cp/pph-in.c View 5 chunks +39 lines, -24 lines 0 comments Download
M gcc/cp/pph-out.c View 1 chunk +9 lines, -1 line 0 comments Download
A gcc/testsuite/g++.dg/pph/a0incomplete2.hi View 1 chunk +12 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/a0incomplete3.hi View 1 chunk +21 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/a0incomplete4.cci View 1 chunk +5 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/x0incomplete1.h View 1 chunk +6 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/x0incomplete2.h View 1 chunk +6 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/x0incomplete3.h View 1 chunk +7 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/x1incomplete3.h View 1 chunk +7 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/x1incomplete4.cc View 1 chunk +2 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/x2incomplete4.cc View 1 chunk +5 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/x3incomplete412.cc View 1 chunk +4 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/x3incomplete413.cc View 1 chunk +3 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/x4incomplete4123.cc View 1 chunk +4 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/x4incomplete4321.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M gcc/testsuite/g++.dg/pph/x4structover1.cc View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 2
Lawrence Crowl
This patch adds minimal type merging. It unifies the existence of types for a struct ...
12 years, 1 month ago (2012-02-02 02:45:01 UTC) #1
Diego Novillo
12 years, 1 month ago (2012-02-02 02:54:02 UTC) #2
On Wed, Feb 1, 2012 at 18:44, Lawrence Crowl <crowl@google.com> wrote:

> @@ -1393,7 +1393,12 @@ dump_function_decl (tree t, int flags)
>     }
>
>   fntype = TREE_TYPE (t);
> -  parmtypes = FUNCTION_FIRST_USER_PARMTYPE (t);
> +  gcc_assert (fntype)

You forgot a semi-colon :)  Fixed.


Diego.
Sign in to reply to this message.

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