9 years, 5 months ago
(2015-10-13 08:15:51 UTC)
#3
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.
9 years, 5 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
Issue 267910043: simple gcc coverage for kernel fuzzing
Created 9 years, 5 months ago by dvyukov
Modified 9 years, 5 months ago
Reviewers: hipphipphurax
Base URL: svn+ssh://gcc.gnu.org/svn/gcc/trunk/gcc/
Comments: 3