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

Issue 267910043: simple gcc coverage for kernel fuzzing

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years, 8 months ago by dvyukov
Modified:
6 years, 7 months ago
Reviewers:
hipphipphurax
Base URL:
svn+ssh://gcc.gnu.org/svn/gcc/trunk/gcc/
Visibility:
Public.

Description

simple gcc coverage for kernel fuzzing

Patch Set 1 #

Total comments: 3

Patch Set 2 : rebase to rev 228818 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+123 lines, -0 lines) Patch
M Makefile.in View 1 2 chunks +2 lines, -0 lines 0 comments Download
M common.opt View 1 1 chunk +4 lines, -0 lines 0 comments Download
A fuzz.c View 1 1 chunk +115 lines, -0 lines 0 comments Download
M passes.def View 1 1 chunk +1 line, -0 lines 0 comments Download
M tree-pass.h View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4
hipphipphurax
Hi, I needed to change the types for two decleration for the my copy of ...
6 years, 7 months ago (2015-10-13 06:33:01 UTC) #1
dvyukov
https://codereview.appspot.com/267910043/diff/1/fuzz.c File fuzz.c (right): https://codereview.appspot.com/267910043/diff/1/fuzz.c#newcode64 fuzz.c:64: gimple stmt, f; On 2015/10/13 06:33:00, hipphipphurax wrote: > ...
6 years, 7 months ago (2015-10-13 07:44:51 UTC) #2
hipphipphurax
https://codereview.appspot.com/267910043/diff/1/fuzz.c File fuzz.c (right): https://codereview.appspot.com/267910043/diff/1/fuzz.c#newcode64 fuzz.c:64: gimple stmt, f; On 2015/10/13 07:44:51, dvyukov wrote: > ...
6 years, 7 months ago (2015-10-13 08:15:51 UTC) #3
dvyukov
6 years, 7 months ago (2015-10-14 18:11:38 UTC) #4
On 2015/10/13 08:15:51, hipphipphurax wrote:
> https://codereview.appspot.com/267910043/diff/1/fuzz.c
> File fuzz.c (right):
> 
> https://codereview.appspot.com/267910043/diff/1/fuzz.c#newcode64
> fuzz.c:64: gimple stmt, f;
> On 2015/10/13 07:44:51, dvyukov wrote:
> > On 2015/10/13 06:33:00, hipphipphurax wrote:
> > > I needed to change those two variables to pointers to get it compiling.
> > 
> > Acknowledged
> > Is your revision older or newer than 227353?
> > Eventually we need to upstream it and then all such problems will go away.
> 
> The last upstream commit contains this
> 
> git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@228707
> 138bc75d-0d04-0410-961f-82ee72b054a4
> 
> I read this as 228707 but not sure.


I've rebased this patch to gcc rev 228818.
And, yes, it required to add starts where you said.
thanks
Sign in to reply to this message.

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