[PATCH v2 1/1] opcodes: microblaze: Add new bit-field instructions
Maciej W. Rozycki
macro@orcam.me.uk
Mon Oct 9 13:32:43 GMT 2023
On Mon, 9 Oct 2023, Frager, Neal wrote:
> > Add test cases for new instructions.
> > Fix GNU Coding Standard issues.
> > Run binutils test suite.
>
> Thank you for feedback. Could you possibly help me with resolving the
> GNU coding standard issues and the build issue Maciej is reporting?
See: <https://www.gnu.org/prep/standards/standards.html#Formatting> and
<https://www.gnu.org/prep/standards/standards.html#Comments> for the
relevant parts of the GNU Coding Standards (you're welcome to review the
whole document).
To reproduce the regressions configure your binutils with the command I
provided and run the testsuite using `make check'. Test logs will be in
binutils/binutils.log, gas/testsuite/gas.log and ld/ld.log (there will be
corresponding .sum files with test summaries). There may be preexisting
issues, but you have the list of failures to watch out for in my previous
message. You need to make sure no new failures have been introduced by
your change.
As I write I came by these issues by chance in unrelated verification
and I won't provide further feedback as I have no interest in the
MicroBlaze port and I cannot afford the time to be distracted. Michael,
the port maintainer may be able to provide you with further guidance.
HTH,
Maciej
More information about the Binutils
mailing list