[PATCH v1 1/1] opcodes: microblaze: Add new bit-field instructions
Frager, Neal
neal.frager@amd.com
Thu Sep 28 09:53:14 GMT 2023
Hi Nick,
> Le 27 sept. 2023 à 17:29, Nick Clifton <nickc@redhat.com> a écrit :
>
> Hi Neal,
>
>> This patches adds new bsefi and bsifi instructions.
>> BSEFI- The instruction shall extract a bit field from a
>> register and place it right-adjusted in the destination register.
>> The other bits in the destination register shall be set to zero.
>> BSIFI- The instruction shall insert a right-adjusted bit field
>> from a register at another position in the destination register.
>> The rest of the bits in the destination register shall be unchanged.
>> Further documentation of these instructions can be found here:
>> https://docs.xilinx.com/v/u/en-US/ug984-vivado-microblaze-ref
>> This patch has been tested for years of AMD Xilinx Yocto
>> releases as part of the following patch set:
>> https://github.com/Xilinx/meta-xilinx/tree/master/meta-microblaze/recipes-devtools/binutils/binutils
>> Signed-off-by:nagaraju <nagaraju.mekala@amd.com>
>> Signed-off-by: Neal Frager <neal.frager@amd.com>
>> ---
>> gas/config/tc-microblaze.c | 71 +++++++++++++++++++++++++++++++++++++-
>> opcodes/microblaze-dis.c | 16 +++++++++
>> opcodes/microblaze-opc.h | 12 ++++++-
>> opcodes/microblaze-opcm.h | 6 +++-
>> 4 files changed, 102 insertions(+), 3 deletions(-)
>
> Patch approved, please apply.
>
> But please may I also request a follow up patch to add a test
> for these new instructions to the assembler testsuite, eg in
> gas/testsuite/gas/microblaze/allinsn.[sd],
>
Please do not apply this patch yet. I would like to make some changes first and submit a v2.
Thank you.
> Cheers
> Nick
>
Best regards,
Neal Frager
AMD
More information about the Binutils
mailing list