Thanks to everyone who reviewed our original patch set.  I've tried to CC
everyone who provided a review, sorry if I missed anyone!

Andrew, Kito and I believe that we've addressed almost all of the feedback from
the reviews of our v1 patch set.  Since it's been a while we wanted to get a v2
patch set out, just to make sure we're all on the same page going forward.
As far as I know, there's just one thing that still needs to be taken care of:

 * I haven't managed to get contrib/config-list.mk to build anywhere, and the
   errors I'm seeing look like valid warnings to me.  One example is the
   following snippit from gcc/genhooks.c:

     while (fscanf (f, "%*[^@]"), buf[0] = '\0',
            fscanf (f, "@%5[^ \n]", buf) != EOF)

   which throws a warning about not checking the return value of the first
   fscanf.  I've tried various GCC releases and the trunk, but this looks like
   a valid warning to me.

   If I'm meant to go through and fix all the warnings I can do that, but it
   might take a while.  Is this what's expected?  I feel like I'm doing
   something wrong here...

Additionally, here's a diff against wwwdocs.  This is really just to check this
is all I'm supposed to do, I can submit a proper patch via the mailing list (I
just don't know how to use CVS, sorry).

  Index: htdocs/gcc-7/changes.html
  ===================================================================
  RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-7/changes.html,v
  retrieving revision 1.54
  diff -u -r1.54 changes.html
  --- htdocs/gcc-7/changes.html   1 Feb 2017 19:23:00 -0000       1.54
  +++ htdocs/gcc-7/changes.html   2 Feb 2017 08:23:20 -0000
  @@ -826,6 +826,11 @@
  
   <!-- <h3 id="hsa">Heterogeneous Systems Architecture</h3> -->
  
  +<h3 id="riscv">RISC-V</h3>
  +<ul>
  +  <li>Support for the RISC-V instruction set has been added</li>
  +</ul>
  +
   <h3 id="x86">IA-32/x86-64</h3>
   <ul>
     <li>Support for the AVX-512 Fused Multiply Accumulation Packed Single 
precision

Thanks for all the reviews!

[PATCH 1/6] RISC-V Port: gcc/config/riscv/riscv.c
[PATCH 2/6] RISC-V Port: gcc
[PATCH 3/6] RISC-V Port: libgcc
[PATCH 4/6] RISC-V Port: libatomic
[PATCH 5/6] RISC-V Port: gcc/testsuite
[PATCH 6/6] RISC-V Port: contrib

Reply via email to