Re: [PATCH 2/2] [WebAssembly] Tests

[Pip Cet <pipcet at gmail dot com>]

On Thu, Mar 30, 2017 at 10:06 AM, Nick Clifton <nickc@redhat.com> wrote:
> Hi Pip,
>
>> This patch adds a number of tests for the WebAssembly GAS backend, as
>> well as a few tests for the WebAssembly BFD backend.
>
> Thanks very much for taking the time to do this.  I have checked your
> patch in, along with a small addition to the readelf program so that
> it can recognise wasm32 binaries, and their relocs.  This allows a
> couple of the generic gas testsuite tests to pass.

Thank you very much! I'll try investigating why the readelf tests
appeared to be passing here. I've attached another patch with two very
minor changes.

Suggested change log entry:
binutils/:
2017-03-30  Pip Cet  <pipcet@gmail.com>

    * NEWS: Use "WebAssembly" consistently.
    * testsuite/binutils-all/wasm32/wasm32.exp: Fix copyright notice.

diff --git a/binutils/NEWS b/binutils/NEWS
index 1b824facae..d9d65c71e4 100644
--- a/binutils/NEWS
+++ b/binutils/NEWS
@@ -1,6 +1,6 @@
 -*- text -*-

-* Add support for the wasm32 ELF conversion of the Web Assembly file format.
+* Add support for the wasm32 ELF conversion of the WebAssembly file format.

 * Add --inlines option to objdump, which extends the --line-numbers option
   so that inlined functions will display their nesting information.
diff --git a/binutils/testsuite/binutils-all/wasm32/wasm32.exp
b/binutils/testsuite/binutils-all/wasm32/wasm32.exp
index b95569a744..af39cd4855 100644
--- a/binutils/testsuite/binutils-all/wasm32/wasm32.exp
+++ b/binutils/testsuite/binutils-all/wasm32/wasm32.exp
@@ -1,5 +1,4 @@
-#   Copyright (C) 2010-2017 Free Software Foundation, Inc.
-#   Copyright (C) 2017 Pip Cet <pipcet@gmail.com>
+#   Copyright (C) 2017 Free Software Foundation, Inc.

 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by

Attachment: binutils-wasm-007.diff
Description: Text document