[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