From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from antelope.elm.relay.mailchannels.net (antelope.elm.relay.mailchannels.net [23.83.212.4]) by sourceware.org (Postfix) with ESMTPS id CD1393858D33 for ; Mon, 9 Oct 2023 14:54:13 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org CD1393858D33 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=eagercon.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=eagercon.com X-Sender-Id: dreamhost|x-authsender|eager@eagerm.com Received: from relay.mailchannels.net (localhost [127.0.0.1]) by relay.mailchannels.net (Postfix) with ESMTP id EF36E4247F; Mon, 9 Oct 2023 14:54:11 +0000 (UTC) Received: from pdx1-sub0-mail-a246.dreamhost.com (unknown [127.0.0.6]) (Authenticated sender: dreamhost) by relay.mailchannels.net (Postfix) with ESMTPA id 5E96442690; Mon, 9 Oct 2023 14:54:10 +0000 (UTC) ARC-Seal: i=1; s=arc-2022; d=mailchannels.net; t=1696863250; a=rsa-sha256; cv=none; b=cxpqxDvRI99kBvp60rpNYmuap8qrEsXH3G+88SbwpchYt506Hl/7yAgMQL5pww5MMiSKHN w2f4J7oYfQ/TGf772+hhQm7PbToWllFide+Oz7EMeFb6x1pEKzzUahTUF1cIQaR1IRtPEe KXNE7Sw6otNvb7mkNgEzGSkhiR/rP8EI0mQpQ9FZacnIJHd64UV01DObjkFlEY0ivR3h4e t2AuLFaV5IPuEqM8NGwuaOuW5JKcHIW9eGYbba28pk0OCcQs28mX39zSwJzjR6Yil0ZFb1 9Fx+t1NVK34kk4jin1zQfFtuGapIPnd3jG/quBwbwKZ94ueGrrvoVRplkQANbw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=mailchannels.net; s=arc-2022; t=1696863250; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=hCtdbxnpveZH0hu+2C4Lom/O0CrxPzELEcmH2k+RFhQ=; b=JT1knF/d/23426yRkwV93bYJeW/uSGmuCmtxZm6RtU6t+IuxiC+WVcUqWNllMZSkV3onQ2 ml2j4z9qYDLtYSFRQcsXpP4Qg4AjebWsOYEM7PK+cnJyLi+hyU8U7obS+j3zliOwpSR4SN ucEa3EpozaE6ym7HFDQgxJTEz03F7pzk37hHE0LdwE+H4GL9Hci4gfrPHYlB6KNqI2EM+p Nj7rVnnH1EyqAoRsHcxQmCLkbHDPmvHgLTNcevzt0PVoUMhqAcvGfz+7wFDSJtwWqFVzQC 9hPxcWMrCUjI9e38shH/ltJEYWIzThyUMoIc2JDDvCA7oioUr8rz+zU3/WC5kQ== ARC-Authentication-Results: i=1; rspamd-7d5dc8fd68-jgnfk; auth=pass smtp.auth=dreamhost smtp.mailfrom=eager@eagercon.com X-Sender-Id: dreamhost|x-authsender|eager@eagerm.com X-MC-Relay: Neutral X-MailChannels-SenderId: dreamhost|x-authsender|eager@eagerm.com X-MailChannels-Auth-Id: dreamhost X-Whimsical-Relation: 092235ce659ff565_1696863251779_296174753 X-MC-Loop-Signature: 1696863251779:3649203053 X-MC-Ingress-Time: 1696863251779 Received: from pdx1-sub0-mail-a246.dreamhost.com (pop.dreamhost.com [64.90.62.162]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384) by 100.126.222.25 (trex/6.9.1); Mon, 09 Oct 2023 14:54:10 +0000 Received: from [192.168.20.10] (c-73-170-238-207.hsd1.ca.comcast.net [73.170.238.207]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) (Authenticated sender: eager@eagerm.com) by pdx1-sub0-mail-a246.dreamhost.com (Postfix) with ESMTPSA id 4S429Y4v1nz71; Mon, 9 Oct 2023 07:54:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=eagercon.com; s=dreamhost; t=1696863250; bh=hCtdbxnpveZH0hu+2C4Lom/O0CrxPzELEcmH2k+RFhQ=; h=Date:Subject:To:Cc:From:Content-Type:Content-Transfer-Encoding; b=qTbUB8quSpWS00LA70czlb5qj7pCWwwuDgksNc54mYx/TrAsHG+TElmTQ7X0+O7S8 w77ywxXucQ/5BbZL3K1Dpvt4we+aiBBnW2sfdQFR0442PPmcrh5D5WuWQH7bqmrkN6 Q3IM6K75h3/z0eX3s8RGSdrzl22ODD+zI4XwF6mng1Wxk2TTAQiuQdq8EAui+5i+4N CAIvdd+ZrnhLffRhuPTDUux62pJJTy7N7BL7BmU5jzBprWcD6z1ev9BWW5wTveZUR5 7UCg/9VjAQ4Ip0D8ej0XMCFBSy3xRTELkG1E2dpfAww40YKJ3N8s9EOQlFBbY6ro+N A13M9dnBZfmdQ== Message-ID: <3c335511-657d-1089-02b1-78be38a0addd@eagercon.com> Date: Mon, 9 Oct 2023 07:54:08 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 Subject: Re: [PATCH v3 1/1] opcodes: microblaze: Add new bit-field instructions Content-Language: en-US To: Neal Frager , binutils@sourceware.org Cc: ibai.erkiaga-elorza@amd.com, nagaraju.mekala@amd.com, mark.hatle@amd.com, sadanand.mutyala@amd.com, appa.rao.nali@amd.com, vidhumouli.hunsigida@amd.com, luca.ceresoli@bootlin.com, nickc@redhat.com, mark@klomp.org, macro@orcam.me.uk References: <20231009125144.377940-1-neal.frager@amd.com> From: Michael Eager In-Reply-To: <20231009125144.377940-1-neal.frager@amd.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-12.7 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,KAM_SHORT,KAM_STOCKGEN,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Neal -- The GNU Coding Standards can be found here: https://www.gnu.org/prep/standards/ I've made some notes in the patch where there are changes needed. On 10/9/23 05:51, Neal Frager wrote: > 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 > Signed-off-by: Ibai Erkiaga > Signed-off-by: Neal Frager > --- > V1->V2: > -corrected relocation values for the linker > V2->V3: > - fixed build issue for 32-bit hosts > - added test cases for bsefi and bsifi instructions > --- > bfd/bfd-in2.h | 5 + > bfd/elf32-microblaze.c | 127 +++++++++++++++++++------ > bfd/libbfd.h | 1 + > bfd/reloc.c | 6 ++ > binutils/readelf.c | 4 + > gas/config/tc-microblaze.c | 77 ++++++++++++++- > gas/testsuite/gas/microblaze/allinsn.d | 6 ++ > gas/testsuite/gas/microblaze/allinsn.s | 9 +- > include/elf/microblaze.h | 1 + > opcodes/microblaze-dis.c | 16 ++++ > opcodes/microblaze-opc.h | 12 ++- > opcodes/microblaze-opcm.h | 6 +- > 12 files changed, 237 insertions(+), 33 deletions(-) > > diff --git a/bfd/bfd-in2.h b/bfd/bfd-in2.h > index eddb9902f5e..468e14af262 100644 > --- a/bfd/bfd-in2.h > +++ b/bfd/bfd-in2.h > @@ -6461,6 +6461,11 @@ value relative to the read-write small data area anchor */ > expressions of the form "Symbol Op Symbol" */ > BFD_RELOC_MICROBLAZE_32_SYM_OP_SYM, > > +/* This is a 32 bit reloc that stores the 32 bit pc relative > +value in two words (with an imm instruction).No relocation is > +done here - only used for relaxing */ > + BFD_RELOC_MICROBLAZE_32_NONE, > + > /* This is a 64 bit reloc that stores the 32 bit pc relative > value in two words (with an imm instruction). No relocation is > done here - only used for relaxing */ > diff --git a/bfd/elf32-microblaze.c b/bfd/elf32-microblaze.c > index a7e81c70fc8..760b3236810 100644 > --- a/bfd/elf32-microblaze.c > +++ b/bfd/elf32-microblaze.c > @@ -174,6 +174,21 @@ static reloc_howto_type microblaze_elf_howto_raw[] = > 0x0000ffff, /* Dest Mask. */ > false), /* PC relative offset? */ > > + /* This reloc does nothing. Used for relaxation. */ > + HOWTO (R_MICROBLAZE_32_NONE, /* Type. */ > + 0, /* Rightshift. */ > + 2, /* Size (0 = byte, 1 = short, 2 = long). */ > + 32, /* Bitsize. */ > + true, /* PC_relative. */ > + 0, /* Bitpos. */ > + complain_overflow_bitfield, /* Complain on overflow. */ > + NULL, /* Special Function. */ > + "R_MICROBLAZE_32_NONE",/* Name. */ > + false, /* Partial Inplace. */ > + 0, /* Source Mask. */ > + 0, /* Dest Mask. */ > + false), /* PC relative offset? */ > + Use leading tabs, not spaces. > /* This reloc does nothing. Used for relaxation. */ > HOWTO (R_MICROBLAZE_64_NONE, /* Type. */ > 0, /* Rightshift. */ > @@ -560,6 +575,9 @@ microblaze_elf_reloc_type_lookup (bfd * abfd ATTRIBUTE_UNUSED, > case BFD_RELOC_NONE: > microblaze_reloc = R_MICROBLAZE_NONE; > break; > + case BFD_RELOC_MICROBLAZE_32_NONE: > + microblaze_reloc = R_MICROBLAZE_32_NONE; > + break; > case BFD_RELOC_MICROBLAZE_64_NONE: > microblaze_reloc = R_MICROBLAZE_64_NONE; > break; > @@ -1954,18 +1972,26 @@ microblaze_elf_relax_section (bfd *abfd, > } > break; > case R_MICROBLAZE_NONE: > + case R_MICROBLAZE_32_NONE: > { > /* This was a PC-relative instruction that was > completely resolved. */ > size_t sfix, efix; > + unsigned int val; > bfd_vma target_address; > target_address = irel->r_addend + irel->r_offset; > sfix = calc_fixup (irel->r_offset, 0, sec); > efix = calc_fixup (target_address, 0, sec); > - irel->r_addend -= (efix - sfix); > - /* Should use HOWTO. */ > - microblaze_bfd_write_imm_value_32 (abfd, contents + irel->r_offset, > - irel->r_addend); > + > + /* Validate the in-band val. */ > + val = bfd_get_32 (abfd, contents + irel->r_offset); > + if (val != irel->r_addend && ELF32_R_TYPE (irel->r_info) == R_MICROBLAZE_32_NONE) { Left brace on next line. > + fprintf(stderr, "%d: CORRUPT relax reloc %x %lx\n", __LINE__, val, (unsigned long) irel->r_addend); Wrap lines which are longer than 80 characters. Function calls have a space between function name (fprintf) and argument list. > + } > + irel->r_addend -= (efix - sfix); > + /* Should use HOWTO. */ > + microblaze_bfd_write_imm_value_32 (abfd, contents + irel->r_offset, > + irel->r_addend); > } > break; > case R_MICROBLAZE_64_NONE: > @@ -2009,31 +2035,74 @@ microblaze_elf_relax_section (bfd *abfd, > irelscanend = irelocs + o->reloc_count; > for (irelscan = irelocs; irelscan < irelscanend; irelscan++) > { > - if (ELF32_R_TYPE (irelscan->r_info) == (int) R_MICROBLAZE_32) > - { > - isym = isymbuf + ELF32_R_SYM (irelscan->r_info); > + if (ELF32_R_TYPE (irelscan->r_info) == (int) R_MICROBLAZE_32_NONE) > + { > + unsigned int val; > + > + isym = isymbuf + ELF32_R_SYM (irelscan->r_info); > + > + /* hax: We only do the following fixup for debug location lists. */ > + if (strcmp(".debug_loc", o->name)) > + continue; > + > + /* This was a PC-relative instruction that was completely resolved. */ > + if (ocontents == NULL) > + { > + if (elf_section_data (o)->this_hdr.contents != NULL) > + ocontents = elf_section_data (o)->this_hdr.contents; > + else > + { > + /* We always cache the section contents. > + Perhaps, if info->keep_memory is FALSE, we > + should free them, if we are permitted to. */ > + > + if (o->rawsize == 0) > + o->rawsize = o->size; > + ocontents = (bfd_byte *) bfd_malloc (o->rawsize); > + if (ocontents == NULL) > + goto error_return; > + if (!bfd_get_section_contents (abfd, o, ocontents, > + (file_ptr) 0, > + o->rawsize)) > + goto error_return; > + elf_section_data (o)->this_hdr.contents = ocontents; > + } > + } > > - /* Look at the reloc only if the value has been resolved. */ > - if (isym->st_shndx == shndx > - && (ELF32_ST_TYPE (isym->st_info) == STT_SECTION)) > - { > - if (ocontents == NULL) > - { > - if (elf_section_data (o)->this_hdr.contents != NULL) > - ocontents = elf_section_data (o)->this_hdr.contents; > - else > - { > - /* We always cache the section contents. > - Perhaps, if info->keep_memory is FALSE, we > - should free them, if we are permitted to. */ > - if (o->rawsize == 0) > - o->rawsize = o->size; > - ocontents = (bfd_byte *) bfd_malloc (o->rawsize); > - if (ocontents == NULL) > - goto error_return; > - if (!bfd_get_section_contents (abfd, o, ocontents, > - (file_ptr) 0, > - o->rawsize)) > + val = bfd_get_32 (abfd, ocontents + irelscan->r_offset); > + if (val != irelscan->r_addend) { > + fprintf(stderr, "%d: CORRUPT relax reloc! %x %lx\n", __LINE__, val, (unsigned long) irelscan->r_addend); > + } > + > + irelscan->r_addend -= calc_fixup (irelscan->r_addend, 0, sec); > + microblaze_bfd_write_imm_value_32 (abfd, ocontents + irelscan->r_offset, > + irelscan->r_addend); > + } > + if (ELF32_R_TYPE (irelscan->r_info) == (int) R_MICROBLAZE_32) > + { > + isym = isymbuf + ELF32_R_SYM (irelscan->r_info); > + > + /* Look at the reloc only if the value has been resolved. */ > + if (isym->st_shndx == shndx > + && (ELF32_ST_TYPE (isym->st_info) == STT_SECTION)) > + { > + if (ocontents == NULL) > + { > + if (elf_section_data (o)->this_hdr.contents != NULL) > + ocontents = elf_section_data (o)->this_hdr.contents; > + else > + { > + /* We always cache the section contents. > + Perhaps, if info->keep_memory is FALSE, we > + should free them, if we are permitted to. */ > + if (o->rawsize == 0) > + o->rawsize = o->size; > + ocontents = (bfd_byte *) bfd_malloc (o->rawsize); > + if (ocontents == NULL) > + goto error_return; > + if (!bfd_get_section_contents (abfd, o, ocontents, > + (file_ptr) 0, > + o->rawsize)) > goto error_return; > elf_section_data (o)->this_hdr.contents = ocontents; > } > @@ -2068,7 +2137,7 @@ microblaze_elf_relax_section (bfd *abfd, > elf_section_data (o)->this_hdr.contents = ocontents; > } > } > - irelscan->r_addend -= calc_fixup (irel->r_addend > + irelscan->r_addend -= calc_fixup (irelscan->r_addend > + isym->st_value, > 0, > sec); > diff --git a/bfd/libbfd.h b/bfd/libbfd.h > index d5f42f22c08..d729dc48e7c 100644 > --- a/bfd/libbfd.h > +++ b/bfd/libbfd.h > @@ -3010,6 +3010,7 @@ static const char *const bfd_reloc_code_real_names[] = { "@@uninitialized@@", > "BFD_RELOC_MICROBLAZE_32_ROSDA", > "BFD_RELOC_MICROBLAZE_32_RWSDA", > "BFD_RELOC_MICROBLAZE_32_SYM_OP_SYM", > + "BFD_RELOC_MICROBLAZE_32_NONE", > "BFD_RELOC_MICROBLAZE_64_NONE", > "BFD_RELOC_MICROBLAZE_64_GOTPC", > "BFD_RELOC_MICROBLAZE_64_GOT", > diff --git a/bfd/reloc.c b/bfd/reloc.c > index 2ac883d0eac..3ea2afc0d4e 100644 > --- a/bfd/reloc.c > +++ b/bfd/reloc.c > @@ -6694,6 +6694,12 @@ ENUM > ENUMDOC > This is a 32 bit reloc for the microblaze to handle > expressions of the form "Symbol Op Symbol" > +ENUM > + BFD_RELOC_MICROBLAZE_32_NONE > +ENUMDOC > + This is a 32 bit reloc that stores the 32 bit pc relative > + value in two words (with an imm instruction). No relocation is > + done here - only used for relaxing > ENUM > BFD_RELOC_MICROBLAZE_64_NONE > ENUMDOC > diff --git a/binutils/readelf.c b/binutils/readelf.c > index c9b6210e229..57ac46e92b2 100644 > --- a/binutils/readelf.c > +++ b/binutils/readelf.c > @@ -15279,6 +15279,10 @@ is_8bit_abs_reloc (Filedata * filedata, unsigned int reloc_type) > return reloc_type == 54; /* R_RISCV_SET8. */ > case EM_Z80: > return reloc_type == 1; /* R_Z80_8. */ > + case EM_MICROBLAZE: > + return reloc_type == 33 /* R_MICROBLAZE_32_NONE. */ > + || reloc_type == 0 /* R_MICROBLAZE_NONE. */ > + || reloc_type == 9; /* R_MICROBLAZE_64_NONE. */ > default: > return false; > } > diff --git a/gas/config/tc-microblaze.c b/gas/config/tc-microblaze.c > index d900a9e1d05..1d378633422 100644 > --- a/gas/config/tc-microblaze.c > +++ b/gas/config/tc-microblaze.c > @@ -915,7 +915,7 @@ md_assemble (char * str) > unsigned reg2; > unsigned reg3; > unsigned isize; > - unsigned int immed = 0, temp; > + unsigned int immed = 0, immed2 = 0, temp; > expressionS exp; > char name[20]; > > @@ -1177,6 +1177,77 @@ md_assemble (char * str) > inst |= (immed << IMM_LOW) & IMM5_MASK; > break; > > + case INST_TYPE_RD_R1_IMM5_IMM5: > + if (strcmp (op_end, "")) > + op_end = parse_reg (op_end + 1, ®1); /* Get rd. */ > + else > + { > + as_fatal (_("Error in statement syntax")); > + reg1 = 0; > + } > + if (strcmp (op_end, "")) > + op_end = parse_reg (op_end + 1, ®2); /* Get r1. */ > + else > + { > + as_fatal (_("Error in statement syntax")); > + reg2 = 0; > + } > + > + /* Check for spl registers. */ > + if (check_spl_reg (®1)) > + as_fatal (_("Cannot use special register with this instruction")); > + if (check_spl_reg (®2)) > + as_fatal (_("Cannot use special register with this instruction")); > + > + /* Width immediate value. */ > + if (strcmp (op_end, "")) > + op_end = parse_imm (op_end + 1, &exp, MIN_IMM_WIDTH, MAX_IMM_WIDTH); > + else > + as_fatal (_("Error in statement syntax")); > + if (exp.X_op != O_constant) > + { > + as_warn (_("Symbol used as immediate width value for bit field instruction")); > + immed = 1; > + } > + else > + immed = exp.X_add_number; > + if (opcode->instr == bsefi && immed > 31) > + as_fatal (_("Width value must be less than 32")); > + > + /* Shift immediate value. */ > + if (strcmp (op_end, "")) > + op_end = parse_imm (op_end + 1, &exp, MIN_IMM, MAX_IMM); > + else > + as_fatal (_("Error in statement syntax")); > + if (exp.X_op != O_constant) > + { > + as_warn (_("Symbol used as immediate shift value for bit field instruction")); > + immed2 = 0; > + } > + else > + { > + output = frag_more (isize); > + immed2 = exp.X_add_number; > + } > + if (immed2 != (immed2 % 32)) > + { > + as_warn (_("Shift value greater than 32. using ")); > + immed2 = immed2 % 32; > + } > + > + /* Check combined value. */ > + if (immed + immed2 > 32) > + as_fatal (_("Width value + shift value must not be greater than 32")); > + > + inst |= (reg1 << RD_LOW) & RD_MASK; > + inst |= (reg2 << RA_LOW) & RA_MASK; > + if (opcode->instr == bsefi) > + inst |= (immed & IMM5_MASK) << IMM_WIDTH_LOW; /* bsefi */ > + else > + inst |= ((immed + immed2 - 1) & IMM5_MASK) << IMM_WIDTH_LOW; /* bsifi */ > + inst |= (immed2 << IMM_LOW) & IMM5_MASK; > + break; > + > case INST_TYPE_R1_R2: > if (strcmp (op_end, "")) > op_end = parse_reg (op_end + 1, ®1); /* Get r1. */ > @@ -2209,9 +2280,12 @@ md_apply_fix (fixS * fixP, > moves code around due to relaxing. */ > if (fixP->fx_r_type == BFD_RELOC_64_PCREL) > fixP->fx_r_type = BFD_RELOC_MICROBLAZE_64_NONE; > + else if (fixP->fx_r_type == BFD_RELOC_32) > + fixP->fx_r_type = BFD_RELOC_MICROBLAZE_32_NONE; > else > fixP->fx_r_type = BFD_RELOC_NONE; > fixP->fx_addsy = section_symbol (absolute_section); > + fixP->fx_done = 0; > } > return; > } > @@ -2432,6 +2506,7 @@ tc_gen_reloc (asection * section ATTRIBUTE_UNUSED, fixS * fixp) > switch (fixp->fx_r_type) > { > case BFD_RELOC_NONE: > + case BFD_RELOC_MICROBLAZE_32_NONE: > case BFD_RELOC_MICROBLAZE_64_NONE: > case BFD_RELOC_32: > case BFD_RELOC_MICROBLAZE_32_LO: > diff --git a/gas/testsuite/gas/microblaze/allinsn.d b/gas/testsuite/gas/microblaze/allinsn.d > index b454fdb30fc..c2cfb0fb528 100644 > --- a/gas/testsuite/gas/microblaze/allinsn.d > +++ b/gas/testsuite/gas/microblaze/allinsn.d > @@ -47,3 +47,9 @@ Disassembly of section .text: > > 00000040 : > 40: 900001e2 swaph r0, r0 > + > +00000044 : > + 44: 64004000 bsefi r0, r0 > + > +00000048 : > + 48: 64008000 bsifi r0, r0 > diff --git a/gas/testsuite/gas/microblaze/allinsn.s b/gas/testsuite/gas/microblaze/allinsn.s > index ffe91ca8eb0..3679c0ceb7c 100644 > --- a/gas/testsuite/gas/microblaze/allinsn.s > +++ b/gas/testsuite/gas/microblaze/allinsn.s > @@ -54,4 +54,11 @@ swapb: > .global swaph > swaph: > swaph r0,r0 > - > + .text > + .global bsefi > +bsefi: > + bsefi r0,r0 > + .text > + .global bsifi > +bsifi: > + bsifi r0,r0 > diff --git a/include/elf/microblaze.h b/include/elf/microblaze.h > index fecdd7e4831..164b36d0978 100644 > --- a/include/elf/microblaze.h > +++ b/include/elf/microblaze.h > @@ -61,6 +61,7 @@ START_RELOC_NUMBERS (elf_microblaze_reloc_type) > RELOC_NUMBER (R_MICROBLAZE_TEXTPCREL_64, 30) /* PC-relative TEXT offset. */ > RELOC_NUMBER (R_MICROBLAZE_TEXTREL_64, 31) /* TEXT Entry offset 64-bit. */ > RELOC_NUMBER (R_MICROBLAZE_TEXTREL_32_LO, 32) /* TEXT Entry offset 32-bit. */ > + RELOC_NUMBER (R_MICROBLAZE_32_NONE, 33) > END_RELOC_NUMBERS (R_MICROBLAZE_max) > > /* Global base address names. */ > diff --git a/opcodes/microblaze-dis.c b/opcodes/microblaze-dis.c > index 12981abfea1..9d602e1d9a9 100644 > --- a/opcodes/microblaze-dis.c > +++ b/opcodes/microblaze-dis.c > @@ -90,6 +90,18 @@ get_field_imm5_mbar (struct string_buf *buf, long instr) > return p; > } > > +static char * > +get_field_imm5width (struct string_buf *buf, long instr) > +{ > + char *p = strbuf (buf); > + > + if (instr & 0x00004000) > + sprintf (p, "%d", (short)(((instr & IMM5_WIDTH_MASK) >> IMM_WIDTH_LOW))); /* bsefi */ > + else > + sprintf (p, "%d", (short)(((instr & IMM5_WIDTH_MASK) >> IMM_WIDTH_LOW) - ((instr & IMM5_MASK) >> IMM_LOW) + 1)); /* bsifi */ Line length. > + return p; > +} > + > static char * > get_field_rfsl (struct string_buf *buf, long instr) > { > @@ -426,6 +438,10 @@ print_insn_microblaze (bfd_vma memaddr, struct disassemble_info * info) > break; > /* For mbar 16 or sleep insn. */ > case INST_TYPE_NONE: > + break; > + /* For bit field insns. */ > + case INST_TYPE_RD_R1_IMM5_IMM5: > + print_func (stream, "\t%s, %s, %s, %s", get_field_rd (&buf, inst),get_field_r1(&buf, inst),get_field_imm5width (&buf, inst), get_field_imm5 (&buf, inst)); Line length. > break; > /* For tuqula instruction */ > case INST_TYPE_RD: > diff --git a/opcodes/microblaze-opc.h b/opcodes/microblaze-opc.h > index 7398e9e246a..96829653488 100644 > --- a/opcodes/microblaze-opc.h > +++ b/opcodes/microblaze-opc.h > @@ -59,6 +59,9 @@ > /* For mbar. */ > #define INST_TYPE_IMM5 20 > > +/* For bsefi and bsifi */ > +#define INST_TYPE_RD_R1_IMM5_IMM5 21 > + > #define INST_TYPE_NONE 25 > > > @@ -89,7 +92,9 @@ > #define OPCODE_MASK_H124 0xFFFF07FF /* High 16, and low 11 bits. */ > #define OPCODE_MASK_H1234 0xFFFFFFFF /* All 32 bits. */ > #define OPCODE_MASK_H3 0xFC000600 /* High 6 bits and bits 21, 22. */ > +#define OPCODE_MASK_H3B 0xFC00C600 /* High 6 bits and bits 16, 17, 21, 22. */ > #define OPCODE_MASK_H32 0xFC00FC00 /* High 6 bits and bit 16-21. */ > +#define OPCODE_MASK_H32B 0xFC00C000 /* High 6 bits and bit 16, 17. */ > #define OPCODE_MASK_H34B 0xFC0000FF /* High 6 bits and low 8 bits. */ > #define OPCODE_MASK_H35B 0xFC0004FF /* High 6 bits and low 9 bits. */ > #define OPCODE_MASK_H34C 0xFC0007E0 /* High 6 bits and bits 21-26. */ > @@ -102,7 +107,7 @@ > #define DELAY_SLOT 1 > #define NO_DELAY_SLOT 0 > > -#define MAX_OPCODES 300 > +#define MAX_OPCODES 301 > > const struct op_code_struct > { > @@ -159,6 +164,8 @@ const struct op_code_struct > {"bslli", INST_TYPE_RD_R1_IMM5, INST_NO_OFFSET, NO_DELAY_SLOT, IMMVAL_MASK_NON_SPECIAL, 0x64000400, OPCODE_MASK_H3, bslli, barrel_shift_inst }, > {"bsrai", INST_TYPE_RD_R1_IMM5, INST_NO_OFFSET, NO_DELAY_SLOT, IMMVAL_MASK_NON_SPECIAL, 0x64000200, OPCODE_MASK_H3, bsrai, barrel_shift_inst }, > {"bsrli", INST_TYPE_RD_R1_IMM5, INST_NO_OFFSET, NO_DELAY_SLOT, IMMVAL_MASK_NON_SPECIAL, 0x64000000, OPCODE_MASK_H3, bsrli, barrel_shift_inst }, > + {"bsefi", INST_TYPE_RD_R1_IMM5_IMM5, INST_NO_OFFSET, NO_DELAY_SLOT, IMMVAL_MASK_NON_SPECIAL, 0x64004000, OPCODE_MASK_H32B, bsefi, barrel_shift_inst }, > + {"bsifi", INST_TYPE_RD_R1_IMM5_IMM5, INST_NO_OFFSET, NO_DELAY_SLOT, IMMVAL_MASK_NON_SPECIAL, 0x64008000, OPCODE_MASK_H32B, bsifi, barrel_shift_inst }, > {"or", INST_TYPE_RD_R1_R2, INST_NO_OFFSET, NO_DELAY_SLOT, IMMVAL_MASK_NON_SPECIAL, 0x80000000, OPCODE_MASK_H4, microblaze_or, logical_inst }, > {"and", INST_TYPE_RD_R1_R2, INST_NO_OFFSET, NO_DELAY_SLOT, IMMVAL_MASK_NON_SPECIAL, 0x84000000, OPCODE_MASK_H4, microblaze_and, logical_inst }, > {"xor", INST_TYPE_RD_R1_R2, INST_NO_OFFSET, NO_DELAY_SLOT, IMMVAL_MASK_NON_SPECIAL, 0x88000000, OPCODE_MASK_H4, microblaze_xor, logical_inst }, > @@ -438,5 +445,8 @@ char pvr_register_prefix[] = "rpvr"; > #define MIN_IMM5 ((int) 0x00000000) > #define MAX_IMM5 ((int) 0x0000001f) > > +#define MIN_IMM_WIDTH ((int) 0x00000001) > +#define MAX_IMM_WIDTH ((int) 0x00000020) > + > #endif /* MICROBLAZE_OPC */ > > diff --git a/opcodes/microblaze-opcm.h b/opcodes/microblaze-opcm.h > index c91b002d951..3c4f8948c76 100644 > --- a/opcodes/microblaze-opcm.h > +++ b/opcodes/microblaze-opcm.h > @@ -29,7 +29,7 @@ enum microblaze_instr > addi, rsubi, addic, rsubic, addik, rsubik, addikc, rsubikc, mul, > mulh, mulhu, mulhsu, swapb, swaph, > idiv, idivu, bsll, bsra, bsrl, get, put, nget, nput, cget, cput, > - ncget, ncput, muli, bslli, bsrai, bsrli, mului, > + ncget, ncput, muli, bslli, bsrai, bsrli, bsefi, bsifi, mului, > /* 'or/and/xor' are C++ keywords. */ > microblaze_or, microblaze_and, microblaze_xor, > andn, pcmpbf, pcmpbc, pcmpeq, pcmpne, sra, src, srl, sext8, sext16, > @@ -130,6 +130,7 @@ enum microblaze_instr_type > #define RB_LOW 11 /* Low bit for RB. */ > #define IMM_LOW 0 /* Low bit for immediate. */ > #define IMM_MBAR 21 /* low bit for mbar instruction. */ > +#define IMM_WIDTH_LOW 6 /* Low bit for immediate width */ > > #define RD_MASK 0x03E00000 > #define RA_MASK 0x001F0000 > @@ -142,6 +143,9 @@ enum microblaze_instr_type > /* Imm mask for mbar. */ > #define IMM5_MBAR_MASK 0x03E00000 > > +/* Imm mask for extract/insert width. */ > +#define IMM5_WIDTH_MASK 0x000007C0 > + > /* FSL imm mask for get, put instructions. */ > #define RFSL_MASK 0x000000F > -- Michael Eager