public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jim Wilson <jim.wilson@linaro.org>
To: "gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>
Subject: [PATCH] [ARM] neon-testgen.ml typo
Date: Thu, 29 Oct 2015 17:29:00 -0000	[thread overview]
Message-ID: <CABXYE2W0SoGRC1HzjEMdXkOBz7LGLvqUFU_tzkED8aOv=X4THw@mail.gmail.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 425 bytes --]

I noticed a comment typo in this file while using grep to look for
other stuff.  The typo is easy to fix.

I tried running neon-testgen.ml to verify, but it is apparently no
longer valid ocaml, as it doesn't work with the ocamlc 4.01.0 I have
on Ubuntu 14.04.  I get a syntax error.  Someone who knows ocaml will
have to fix this.  Meanwhile, the patch to fix the typo should still
be OK, as this is a separate problem.

Jim

[-- Attachment #2: neon-testgen.patch --]
[-- Type: text/x-patch, Size: 778 bytes --]

2015-10-29  Jim Wilson  <jim.wilson@linaro.org>

	* config/arm/neon-testgen.ml: Fix comment typo.

Index: config/arm/neon-testgen.ml
===================================================================
--- config/arm/neon-testgen.ml	(revision 229504)
+++ config/arm/neon-testgen.ml	(working copy)
@@ -76,7 +76,7 @@ let emit_variables chan c_types features spaces =
     match c_types with
       (_, return_ty) :: tys ->
         if return_ty <> "void" then begin
-          (* The intrinsic returns a value.  We need to do explict register
+          (* The intrinsic returns a value.  We need to do explicit register
              allocation for vget_low tests or they fail because of copy
              elimination.  *)
           ((if List.mem Fixed_vector_reg features then

             reply	other threads:[~2015-10-29 17:24 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-29 17:29 Jim Wilson [this message]
2015-10-30 14:37 ` Ramana Radhakrishnan
2015-10-30 14:47   ` Christophe Lyon
2015-11-04 12:16     ` Ramana Radhakrishnan
2015-11-06 20:29       ` Christophe Lyon
2015-11-12 22:18         ` Christophe Lyon
2015-11-13 10:24           ` Christophe Lyon
2015-11-13 10:28             ` Kyrill Tkachov
2015-11-13 11:18             ` Ramana Radhakrishnan
2015-11-13 11:24               ` Kyrill Tkachov
2015-11-13 13:36                 ` Christophe Lyon
2015-11-24 15:25                   ` Christophe Lyon
2015-11-02 12:06 ` Julian Brown
2015-11-02 12:21   ` Kyrill Tkachov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CABXYE2W0SoGRC1HzjEMdXkOBz7LGLvqUFU_tzkED8aOv=X4THw@mail.gmail.com' \
    --to=jim.wilson@linaro.org \
    --cc=gcc-patches@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).