[PATCH] [PowerPC VLE] Add LSP instructions support
Alan Modra
amodra@gmail.com
Mon Aug 21 09:37:00 GMT 2017
On Sun, Aug 20, 2017 at 12:15:11AM +0300, Alexander Fedotov wrote:
> This patch adds LSP instructions support to PowerPC VLE. Based against
> current master branch.
This is OK with a few fixes.
> +2017-08-19 Alexander Fedotov <alexander.fedotov@nxp.com>
> + Edmar Wienskoski <edmar.wienskoski@nxp.com
> +
> + *gas/testsuite/gas/ppc/lsp-checks.d: New test
> + *gas/testsuite/gas/ppc/lsp-checks.l: New
> + *gas/testsuite/gas/ppc/lsp-checks.s: New
> + *gas/testsuite/gas/ppc/lsp.d: New
> + *gas/testsuite/gas/ppc/lsp.s: New
> + *gas/testsuite/gas/ppc/ppc.exp: Run new tests.
> +
Files in a ChangeLog should be relative to the ChangeLog, and there
should be a space after the '*'.
> +static unsigned long
> +insert_evuimm2_ex0 (unsigned long insn,
> + long value,
> + ppc_cpu_t dialect ATTRIBUTE_UNUSED,
> + const char **errmsg)
Formatting here and on other new functions is broken. It should be
static unsigned long
insert_evuimm2_ex0 (unsigned long insn,
long value,
ppc_cpu_t dialect ATTRIBUTE_UNUSED,
const char **errmsg)
Maybe the tabs were eaten by your email client? There are other minor
whitespace issue too. If you add the following to you binutils
.git/config under the [core] heading, git will show them to you
whitespace = indent-with-non-tab,space-before-tab,trailing-space
--
Alan Modra
Australia Development Lab, IBM
More information about the Binutils
mailing list