Apply Dmitry Diky's patches to add relaxation to msp430.
authorNick Clifton <nickc@redhat.com>
Wed, 25 Aug 2004 12:54:15 +0000 (12:54 +0000)
committerNick Clifton <nickc@redhat.com>
Wed, 25 Aug 2004 12:54:15 +0000 (12:54 +0000)
16 files changed:
bfd/ChangeLog
bfd/bfd-in2.h
bfd/elf32-msp430.c
bfd/libbfd.h
bfd/reloc.c
gas/ChangeLog
gas/config/tc-msp430.c
gas/config/tc-msp430.h
gas/doc/c-msp430.texi
include/elf/ChangeLog
include/elf/msp430.h
include/opcode/ChangeLog
include/opcode/msp430.h
ld/ChangeLog
ld/emulparams/msp430all.sh
ld/scripttempl/elf32msp430.sc

index 1aec86c998035a58bc8dd8e5e74e03180bbb3a14..a0a9fc43366559cd78eb675f6a17f69669e430b2 100644 (file)
@@ -1,3 +1,16 @@
+2004-08-25  Dmitry Diky  <diwil@spec.ru>
+
+       * reloc.c (BFD_RELOC_MSP430_2X_PCREL,BFD_RELOC_MSP430_RL_PCREL):
+       Add new relocations.
+       * bfd-in2.h: Regenerate.
+       * libbfd.h: Regenerate..
+       * elf32-msp430.c: Clean-up code.
+       (elf_msp430_howto_table): Add new relocation entries.
+       (bfd_elf32_bfd_reloc_type_lookup): New relocation handlers.
+       (msp430_elf_relax_section): New function.
+       (msp430_elf_relax_delete_bytes): New function.
+       (msp430_elf_symbol_address_p): New function.
+
 2004-08-24  Kazuhiro Inaoka  <inaoka.kazuhiro@renesas.com>
 
        * elf32-m32r.c (m32r_elf_relocate_section): Handle
index 01d6331193caae6ff90b9e1d0db652b669b05904..ddcef0145fcd74e36ed62b2312201b6a4bcb9a7f 100644 (file)
@@ -3587,6 +3587,8 @@ This is the 5 bits of a value.  */
   BFD_RELOC_MSP430_16,
   BFD_RELOC_MSP430_16_PCREL_BYTE,
   BFD_RELOC_MSP430_16_BYTE,
+  BFD_RELOC_MSP430_2X_PCREL,
+  BFD_RELOC_MSP430_RL_PCREL,
 
 /* IQ2000 Relocations.  */
   BFD_RELOC_IQ2000_OFFSET_16,
index dfe33bd246fbf4fbad78a1c2bc21fb3a6cf51284..ba59cf7f874f37db74c6d49dd07a3a27af58fa6a 100644 (file)
 #include "elf-bfd.h"
 #include "elf/msp430.h"
 
-static reloc_howto_type *bfd_elf32_bfd_reloc_type_lookup
-  PARAMS ((bfd *, bfd_reloc_code_real_type));
-
-static void msp430_info_to_howto_rela
-  PARAMS ((bfd *, arelent *, Elf_Internal_Rela *));
-
-static asection *elf32_msp430_gc_mark_hook
-  PARAMS ((asection *, struct bfd_link_info *, Elf_Internal_Rela *,
-          struct elf_link_hash_entry *, Elf_Internal_Sym *));
-
-static bfd_boolean elf32_msp430_gc_sweep_hook
-  PARAMS ((bfd *, struct bfd_link_info *, asection *,
-          const Elf_Internal_Rela *));
-
-static bfd_boolean elf32_msp430_check_relocs
-  PARAMS ((bfd *, struct bfd_link_info *, asection *,
-          const Elf_Internal_Rela *));
-
-static bfd_reloc_status_type msp430_final_link_relocate
-  PARAMS ((reloc_howto_type *, bfd *, asection *, bfd_byte *,
-          Elf_Internal_Rela *, bfd_vma));
-
-static bfd_boolean elf32_msp430_relocate_section
-  PARAMS ((bfd *, struct bfd_link_info *, bfd *, asection *, bfd_byte *,
-          Elf_Internal_Rela *, Elf_Internal_Sym *, asection **));
-
-static void bfd_elf_msp430_final_write_processing
-  PARAMS ((bfd *, bfd_boolean));
-
-static bfd_boolean elf32_msp430_object_p
-  PARAMS ((bfd *));
-
-static void elf32_msp430_post_process_headers
-  PARAMS ((bfd *, struct bfd_link_info *));
-
 /* Use RELA instead of REL.  */
 #undef USE_REL
 
@@ -71,7 +36,7 @@ static reloc_howto_type elf_msp430_howto_table[] =
         32,                    /* bitsize */
         FALSE,                 /* pc_relative */
         0,                     /* bitpos */
-        complain_overflow_bitfield,    /* complain_on_overflow */
+        complain_overflow_bitfield,/* complain_on_overflow */
         bfd_elf_generic_reloc, /* special_function */
         "R_MSP430_NONE",       /* name */
         FALSE,                 /* partial_inplace */
@@ -85,7 +50,7 @@ static reloc_howto_type elf_msp430_howto_table[] =
         32,                    /* bitsize */
         FALSE,                 /* pc_relative */
         0,                     /* bitpos */
-        complain_overflow_bitfield,    /* complain_on_overflow */
+        complain_overflow_bitfield,/* complain_on_overflow */
         bfd_elf_generic_reloc, /* special_function */
         "R_MSP430_32",         /* name */
         FALSE,                 /* partial_inplace */
@@ -100,7 +65,7 @@ static reloc_howto_type elf_msp430_howto_table[] =
         10,                    /* bitsize */
         TRUE,                  /* pc_relative */
         0,                     /* bitpos */
-        complain_overflow_bitfield,    /* complain_on_overflow */
+        complain_overflow_bitfield,/* complain_on_overflow */
         bfd_elf_generic_reloc, /* special_function */
         "R_MSP430_13_PCREL",   /* name */
         FALSE,                 /* partial_inplace */
@@ -119,7 +84,7 @@ static reloc_howto_type elf_msp430_howto_table[] =
         bfd_elf_generic_reloc, /* special_function */
         "R_MSP430_16",         /* name */
         FALSE,                 /* partial_inplace */
-        0xffff,                /* src_mask */
+        0,                     /* src_mask */
         0xffff,                /* dst_mask */
         FALSE),                /* pcrel_offset */
 
@@ -134,7 +99,7 @@ static reloc_howto_type elf_msp430_howto_table[] =
         bfd_elf_generic_reloc, /* special_function */
         "R_MSP430_16_PCREL",   /* name */
         FALSE,                 /* partial_inplace */
-        0xffff,                /* src_mask */
+        0,                     /* src_mask */
         0xffff,                /* dst_mask */
         TRUE),                 /* pcrel_offset */
 
@@ -162,10 +127,40 @@ static reloc_howto_type elf_msp430_howto_table[] =
         0,                     /* bitpos */
         complain_overflow_dont,/* complain_on_overflow */
         bfd_elf_generic_reloc, /* special_function */
-        "R_MSP430_16_PCREL_BYTE",      /* name */
+        "R_MSP430_16_PCREL_BYTE",/* name */
         FALSE,                 /* partial_inplace */
         0xffff,                /* src_mask */
         0xffff,                /* dst_mask */
+        TRUE),                 /* pcrel_offset */
+
+  /* A 13 bit PC relative relocation for complicated polymorphs.  */
+  HOWTO (R_MSP430_2X_PCREL,    /* type */
+        1,                     /* rightshift */
+        2,                     /* size (0 = byte, 1 = short, 2 = long) */
+        10,                    /* bitsize */
+        TRUE,                  /* pc_relative */
+        0,                     /* bitpos */
+        complain_overflow_bitfield,/* complain_on_overflow */
+        bfd_elf_generic_reloc, /* special_function */
+        "R_MSP430_2X_PCREL",   /* name */
+        FALSE,                 /* partial_inplace */
+        0xfff,                 /* src_mask */
+        0xfff,                 /* dst_mask */
+        TRUE),                 /* pcrel_offset */
+
+  /* A 16 bit relaxable relocation for command address.  */
+  HOWTO (R_MSP430_RL_PCREL,    /* type */
+        1,                     /* rightshift */
+        1,                     /* size (0 = byte, 1 = short, 2 = long) */
+        16,                    /* bitsize */
+        TRUE,                  /* pc_relative */
+        0,                     /* bitpos */
+        complain_overflow_dont,/* complain_on_overflow */
+        bfd_elf_generic_reloc, /* special_function */
+        "R_MSP430_RL_PCREL",   /* name */
+        FALSE,                 /* partial_inplace */
+        0,                     /* src_mask */
+        0xffff,                /* dst_mask */
         TRUE)                  /* pcrel_offset */
 };
 
@@ -179,20 +174,21 @@ struct msp430_reloc_map
 
 static const struct msp430_reloc_map msp430_reloc_map[] =
   {
-    {BFD_RELOC_NONE, R_MSP430_NONE},
-    {BFD_RELOC_32, R_MSP430_32},
-    {BFD_RELOC_MSP430_10_PCREL, R_MSP430_10_PCREL},
-    {BFD_RELOC_16, R_MSP430_16_BYTE},
-    {BFD_RELOC_MSP430_16_PCREL, R_MSP430_16_PCREL},
-    {BFD_RELOC_MSP430_16, R_MSP430_16},
+    {BFD_RELOC_NONE,                 R_MSP430_NONE},
+    {BFD_RELOC_32,                   R_MSP430_32},
+    {BFD_RELOC_MSP430_10_PCREL,      R_MSP430_10_PCREL},
+    {BFD_RELOC_16,                   R_MSP430_16_BYTE},
+    {BFD_RELOC_MSP430_16_PCREL,      R_MSP430_16_PCREL},
+    {BFD_RELOC_MSP430_16,            R_MSP430_16},
     {BFD_RELOC_MSP430_16_PCREL_BYTE, R_MSP430_16_PCREL_BYTE},
-    {BFD_RELOC_MSP430_16_BYTE, R_MSP430_16_BYTE}
+    {BFD_RELOC_MSP430_16_BYTE,       R_MSP430_16_BYTE},
+    {BFD_RELOC_MSP430_2X_PCREL,      R_MSP430_2X_PCREL},
+    {BFD_RELOC_MSP430_RL_PCREL,      R_MSP430_RL_PCREL}
   };
 
 static reloc_howto_type *
-bfd_elf32_bfd_reloc_type_lookup (abfd, code)
-     bfd *abfd ATTRIBUTE_UNUSED;
-     bfd_reloc_code_real_type code;
+bfd_elf32_bfd_reloc_type_lookup (bfd * abfd ATTRIBUTE_UNUSED,
+                                bfd_reloc_code_real_type code)
 {
   unsigned int i;
 
@@ -206,10 +202,9 @@ bfd_elf32_bfd_reloc_type_lookup (abfd, code)
 /* Set the howto pointer for an MSP430 ELF reloc.  */
 
 static void
-msp430_info_to_howto_rela (abfd, cache_ptr, dst)
-     bfd *abfd ATTRIBUTE_UNUSED;
-     arelent *cache_ptr;
-     Elf_Internal_Rela *dst;
+msp430_info_to_howto_rela (bfd * abfd ATTRIBUTE_UNUSED,
+                          arelent * cache_ptr,
+                          Elf_Internal_Rela * dst)
 {
   unsigned int r_type;
 
@@ -219,12 +214,11 @@ msp430_info_to_howto_rela (abfd, cache_ptr, dst)
 }
 
 static asection *
-elf32_msp430_gc_mark_hook (sec, info, rel, h, sym)
-     asection *sec;
-     struct bfd_link_info *info ATTRIBUTE_UNUSED;
-     Elf_Internal_Rela *rel;
-     struct elf_link_hash_entry *h;
-     Elf_Internal_Sym *sym;
+elf32_msp430_gc_mark_hook (asection * sec,
+                          struct bfd_link_info * info ATTRIBUTE_UNUSED,
+                          Elf_Internal_Rela * rel,
+                          struct elf_link_hash_entry * h,
+                          Elf_Internal_Sym * sym)
 {
   if (h != NULL)
     {
@@ -252,11 +246,10 @@ elf32_msp430_gc_mark_hook (sec, info, rel, h, sym)
 }
 
 static bfd_boolean
-elf32_msp430_gc_sweep_hook (abfd, info, sec, relocs)
-     bfd *abfd ATTRIBUTE_UNUSED;
-     struct bfd_link_info *info ATTRIBUTE_UNUSED;
-     asection *sec ATTRIBUTE_UNUSED;
-     const Elf_Internal_Rela *relocs ATTRIBUTE_UNUSED;
+elf32_msp430_gc_sweep_hook (bfd * abfd ATTRIBUTE_UNUSED,
+                           struct bfd_link_info * info ATTRIBUTE_UNUSED,
+                           asection * sec ATTRIBUTE_UNUSED,
+                           const Elf_Internal_Rela * relocs ATTRIBUTE_UNUSED)
 {
   /* We don't use got and plt entries for msp430.  */
   return TRUE;
@@ -267,11 +260,8 @@ elf32_msp430_gc_sweep_hook (abfd, info, sec, relocs)
    virtual table relocs for gc.  */
 
 static bfd_boolean
-elf32_msp430_check_relocs (abfd, info, sec, relocs)
-     bfd *abfd;
-     struct bfd_link_info *info;
-     asection *sec;
-     const Elf_Internal_Rela *relocs;
+elf32_msp430_check_relocs (bfd * abfd, struct bfd_link_info * info,
+                          asection * sec, const Elf_Internal_Rela * relocs)
 {
   Elf_Internal_Shdr *symtab_hdr;
   struct elf_link_hash_entry **sym_hashes, **sym_hashes_end;
@@ -308,14 +298,9 @@ elf32_msp430_check_relocs (abfd, info, sec, relocs)
    routines, but a few relocs, we have to do them ourselves.  */
 
 static bfd_reloc_status_type
-msp430_final_link_relocate (howto, input_bfd, input_section,
-                           contents, rel, relocation)
-     reloc_howto_type *howto;
-     bfd *input_bfd;
-     asection *input_section;
-     bfd_byte *contents;
-     Elf_Internal_Rela *rel;
-     bfd_vma relocation;
+msp430_final_link_relocate (reloc_howto_type * howto, bfd * input_bfd,
+                           asection * input_section, bfd_byte * contents,
+                           Elf_Internal_Rela * rel, bfd_vma relocation)
 {
   bfd_reloc_status_type r = bfd_reloc_ok;
   bfd_vma x;
@@ -347,7 +332,37 @@ msp430_final_link_relocate (howto, input_bfd, input_section,
       bfd_put_16 (input_bfd, x, contents);
       break;
 
+    case R_MSP430_2X_PCREL:
+      contents += rel->r_offset;
+      srel = (bfd_signed_vma) relocation;
+      srel += rel->r_addend;
+      srel -= rel->r_offset;
+      srel -= 2;               /* Branch instructions add 2 to the PC...  */
+      srel -= (input_section->output_section->vma +
+              input_section->output_offset);
+
+      if (srel & 1)
+       return bfd_reloc_outofrange;
+
+      /* MSP430 addresses commands as words.  */
+      srel >>= 1;
+
+      /* Check for an overflow.  */
+      if (srel < -512 || srel > 511)
+       return bfd_reloc_overflow;
+
+      x = bfd_get_16 (input_bfd, contents);
+      x = (x & 0xfc00) | (srel & 0x3ff);
+      bfd_put_16 (input_bfd, x, contents);
+      /* Handle second jump instruction.  */
+      x = bfd_get_16 (input_bfd, contents - 2);
+      srel += 1;
+      x = (x & 0xfc00) | (srel & 0x3ff);
+      bfd_put_16 (input_bfd, x, contents - 2);
+      break;
+
     case R_MSP430_16_PCREL:
+    case R_MSP430_RL_PCREL:
       contents += rel->r_offset;
       srel = (bfd_signed_vma) relocation;
       srel += rel->r_addend;
@@ -404,16 +419,14 @@ msp430_final_link_relocate (howto, input_bfd, input_section,
 /* Relocate an MSP430 ELF section.  */
 
 static bfd_boolean
-elf32_msp430_relocate_section (output_bfd, info, input_bfd, input_section,
-                              contents, relocs, local_syms, local_sections)
-     bfd *output_bfd ATTRIBUTE_UNUSED;
-     struct bfd_link_info *info;
-     bfd *input_bfd;
-     asection *input_section;
-     bfd_byte *contents;
-     Elf_Internal_Rela *relocs;
-     Elf_Internal_Sym *local_syms;
-     asection **local_sections;
+elf32_msp430_relocate_section (bfd * output_bfd ATTRIBUTE_UNUSED,
+                              struct bfd_link_info * info,
+                              bfd * input_bfd,
+                              asection * input_section,
+                              bfd_byte * contents,
+                              Elf_Internal_Rela * relocs,
+                              Elf_Internal_Sym * local_syms,
+                              asection ** local_sections)
 {
   Elf_Internal_Shdr *symtab_hdr;
   struct elf_link_hash_entry **sym_hashes;
@@ -520,9 +533,8 @@ elf32_msp430_relocate_section (output_bfd, info, input_bfd, input_section,
    number.  */
 
 static void
-bfd_elf_msp430_final_write_processing (abfd, linker)
-     bfd *abfd;
-     bfd_boolean linker ATTRIBUTE_UNUSED;
+bfd_elf_msp430_final_write_processing (bfd * abfd,
+                                      bfd_boolean linker ATTRIBUTE_UNUSED)
 {
   unsigned long val;
 
@@ -594,8 +606,7 @@ bfd_elf_msp430_final_write_processing (abfd, linker)
 /* Set the right machine number.  */
 
 static bfd_boolean
-elf32_msp430_object_p (abfd)
-     bfd *abfd;
+elf32_msp430_object_p (bfd * abfd)
 {
   int e_set = bfd_mach_msp14;
 
@@ -669,11 +680,10 @@ elf32_msp430_object_p (abfd)
 }
 
 static void
-elf32_msp430_post_process_headers (abfd, link_info)
-     bfd *abfd;
-     struct bfd_link_info *link_info ATTRIBUTE_UNUSED;
+elf32_msp430_post_process_headers (bfd * abfd,
+                                  struct bfd_link_info * link_info ATTRIBUTE_UNUSED)
 {
-  Elf_Internal_Ehdr *i_ehdrp;  /* ELF file header, internal form.  */
+  Elf_Internal_Ehdr * i_ehdrp; /* ELF file header, internal form.  */
 
   i_ehdrp = elf_elfheader (abfd);
 
@@ -684,6 +694,466 @@ elf32_msp430_post_process_headers (abfd, link_info)
   i_ehdrp->e_ident[EI_OSABI] = ELFOSABI_STANDALONE;
 }
 
+/* These functions handle relaxing for the msp430.
+   Relaxation required only in two cases:
+    - Bad hand coding like jumps from one section to another or
+      from file to file.
+    - Sibling calls. This will affect onlu 'jump label' polymorph. Without
+      relaxing this enlarges code by 2 bytes. Sibcalls implemented but
+      do not work in gcc's port by the reason I do not know.
+   Anyway, if a relaxation required, user should pass -relax option to the
+   linker.
+
+   There are quite a few relaxing opportunities available on the msp430:
+
+   ================================================================
+
+   1. 3 words -> 1 word
+
+   eq      ==      jeq label                   jne +4; br lab
+   ne      !=      jne label                   jeq +4; br lab
+   lt      <       jl  label                   jge +4; br lab
+   ltu     <       jlo label                   lhs +4; br lab
+   ge      >=      jge label                   jl  +4; br lab
+   geu     >=      jhs label                   jlo +4; br lab
+
+   2. 4 words -> 1 word
+
+   ltn     <       jn                      jn  +2; jmp +4; br lab
+
+   3. 4 words -> 2 words
+
+   gt      >       jeq +2; jge label       jeq +6; jl  +4; br label
+   gtu     >       jeq +2; jhs label       jeq +6; jlo +4; br label
+
+   4. 4 words -> 2 words and 2 labels
+
+   leu     <=      jeq label; jlo label    jeq +2; jhs +4; br label
+   le      <=      jeq label; jl  label    jeq +2; jge +4; br label
+   =================================================================
+
+   codemap for first cases is (labels masked ):
+             eq:       0x2002,0x4010,0x0000 -> 0x2400
+             ne:       0x2402,0x4010,0x0000 -> 0x2000
+             lt:       0x3402,0x4010,0x0000 -> 0x3800
+             ltu:      0x2c02,0x4010,0x0000 -> 0x2800
+             ge:       0x3802,0x4010,0x0000 -> 0x3400
+             geu:      0x2802,0x4010,0x0000 -> 0x2c00
+
+  second case:
+             ltn:      0x3001,0x3c02,0x4010,0x0000 -> 0x3000
+
+  third case:
+             gt:       0x2403,0x3802,0x4010,0x0000 -> 0x2401,0x3400
+             gtu:      0x2403,0x2802,0x4010,0x0000 -> 0x2401,0x2c00
+
+  fourth case:
+             leu:      0x2401,0x2c02,0x4010,0x0000 -> 0x2400,0x2800
+             le:       0x2401,0x3402,0x4010,0x0000 -> 0x2400,0x3800
+
+  Unspecified case :)
+             jump:     0x4010,0x0000 -> 0x3c00.  */
+
+#define NUMB_RELAX_CODES       12
+static struct rcodes_s
+{
+  int f0, f1;                  /* From code.  */
+  int t0, t1;                  /* To code.  */
+  int labels;                  /* Position of labels: 1 - one label at first
+                                  word, 2 - one at second word, 3 - two
+                                  labels at both.  */
+  int cdx;                     /* Words to match.  */
+  int bs;                      /* Shrink bytes.  */
+  int off;                     /* Offset from old label for new code.  */
+  int ncl;                     /* New code length.  */
+} rcode[] =
+{/*                               lab,cdx,bs,off,ncl */
+  { 0x0000, 0x0000, 0x3c00, 0x0000, 1, 0, 2, 2,         2},    /* jump */
+  { 0x0000, 0x2002, 0x2400, 0x0000, 1, 1, 4, 4,         2},    /* eq */
+  { 0x0000, 0x2402, 0x2000, 0x0000, 1, 1, 4, 4,         2},    /* ne */
+  { 0x0000, 0x3402, 0x3800, 0x0000, 1, 1, 4, 4,         2},    /* lt */
+  { 0x0000, 0x2c02, 0x2800, 0x0000, 1, 1, 4, 4,         2},    /* ltu */
+  { 0x0000, 0x3802, 0x3400, 0x0000, 1, 1, 4, 4,         2},    /* ge */
+  { 0x0000, 0x2802, 0x2c00, 0x0000, 1, 1, 4, 4,         2},    /* geu */
+  { 0x3001, 0x3c02, 0x3000, 0x0000, 1, 2, 6, 6,         2},    /* ltn */
+  { 0x2403, 0x3802, 0x2401, 0x3400, 2, 2, 4, 6,         4},    /* gt */
+  { 0x2403, 0x2802, 0x2401, 0x2c00, 2, 2, 4, 6,         4},    /* gtu */
+  { 0x2401, 0x2c02, 0x2400, 0x2800, 3, 2, 4, 6,         4},    /* leu , 2 labels */
+  { 0x2401, 0x2c02, 0x2400, 0x2800, 3, 2, 4, 6,         4},    /* le  , 2 labels */
+  { 0,             0,      0,      0,      0, 0, 0, 0,  0}
+};
+
+/* Return TRUE if a symbol exists at the given address.  */
+
+static bfd_boolean
+msp430_elf_symbol_address_p (bfd * abfd,
+                            asection * sec,
+                            Elf_Internal_Sym * isym,
+                            bfd_vma addr)
+{
+  Elf_Internal_Shdr *symtab_hdr;
+  unsigned int sec_shndx;
+  Elf_Internal_Sym *isymend;
+  struct elf_link_hash_entry **sym_hashes;
+  struct elf_link_hash_entry **end_hashes;
+  unsigned int symcount;
+
+  sec_shndx = _bfd_elf_section_from_bfd_section (abfd, sec);
+
+  /* Examine all the local symbols.  */
+  symtab_hdr = &elf_tdata (abfd)->symtab_hdr;
+  for (isymend = isym + symtab_hdr->sh_info; isym < isymend; isym++)
+    if (isym->st_shndx == sec_shndx && isym->st_value == addr)
+      return TRUE;
+
+  symcount = (symtab_hdr->sh_size / sizeof (Elf32_External_Sym)
+             - symtab_hdr->sh_info);
+  sym_hashes = elf_sym_hashes (abfd);
+  end_hashes = sym_hashes + symcount;
+  for (; sym_hashes < end_hashes; sym_hashes++)
+    {
+      struct elf_link_hash_entry *sym_hash = *sym_hashes;
+
+      if ((sym_hash->root.type == bfd_link_hash_defined
+          || sym_hash->root.type == bfd_link_hash_defweak)
+         && sym_hash->root.u.def.section == sec
+         && sym_hash->root.u.def.value == addr)
+       return TRUE;
+    }
+
+  return FALSE;
+}
+
+/* Delete some bytes from a section while relaxing.  */
+
+static bfd_boolean
+msp430_elf_relax_delete_bytes (bfd * abfd, asection * sec, bfd_vma addr,
+                              int count)
+{
+  Elf_Internal_Shdr *symtab_hdr;
+  unsigned int sec_shndx;
+  bfd_byte *contents;
+  Elf_Internal_Rela *irel;
+  Elf_Internal_Rela *irelend;
+  Elf_Internal_Rela *irelalign;
+  bfd_vma toaddr;
+  Elf_Internal_Sym *isym;
+  Elf_Internal_Sym *isymend;
+  struct elf_link_hash_entry **sym_hashes;
+  struct elf_link_hash_entry **end_hashes;
+  unsigned int symcount;
+
+  sec_shndx = _bfd_elf_section_from_bfd_section (abfd, sec);
+
+  contents = elf_section_data (sec)->this_hdr.contents;
+
+  /* The deletion must stop at the next ALIGN reloc for an aligment
+     power larger than the number of bytes we are deleting.  */
+
+  irelalign = NULL;
+  toaddr = sec->size;
+
+  irel = elf_section_data (sec)->relocs;
+  irelend = irel + sec->reloc_count;
+
+  /* Actually delete the bytes.  */
+  memmove (contents + addr, contents + addr + count,
+          (size_t) (toaddr - addr - count));
+  sec->size -= count;
+
+  /* Adjust all the relocs.  */
+  for (irel = elf_section_data (sec)->relocs; irel < irelend; irel++)
+    /* Get the new reloc address.  */
+    if ((irel->r_offset > addr && irel->r_offset < toaddr))
+      irel->r_offset -= count;
+
+  /* Adjust the local symbols defined in this section.  */
+  symtab_hdr = & elf_tdata (abfd)->symtab_hdr;
+  isym = (Elf_Internal_Sym *) symtab_hdr->contents;
+  for (isymend = isym + symtab_hdr->sh_info; isym < isymend; isym++)
+    if (isym->st_shndx == sec_shndx
+       && isym->st_value > addr && isym->st_value < toaddr)
+      isym->st_value -= count;
+
+  /* Now adjust the global symbols defined in this section.  */
+  symcount = (symtab_hdr->sh_size / sizeof (Elf32_External_Sym)
+             - symtab_hdr->sh_info);
+  sym_hashes = elf_sym_hashes (abfd);
+  end_hashes = sym_hashes + symcount;
+  for (; sym_hashes < end_hashes; sym_hashes++)
+    {
+      struct elf_link_hash_entry *sym_hash = *sym_hashes;
+
+      if ((sym_hash->root.type == bfd_link_hash_defined
+          || sym_hash->root.type == bfd_link_hash_defweak)
+         && sym_hash->root.u.def.section == sec
+         && sym_hash->root.u.def.value > addr
+         && sym_hash->root.u.def.value < toaddr)
+       sym_hash->root.u.def.value -= count;
+    }
+
+  return TRUE;
+}
+
+
+static bfd_boolean
+msp430_elf_relax_section (bfd * abfd, asection * sec,
+                         struct bfd_link_info * link_info,
+                         bfd_boolean * again)
+{
+  Elf_Internal_Shdr * symtab_hdr;
+  Elf_Internal_Rela * internal_relocs;
+  Elf_Internal_Rela * irel;
+  Elf_Internal_Rela * irelend;
+  bfd_byte *          contents = NULL;
+  Elf_Internal_Sym *  isymbuf = NULL;
+
+  /* Assume nothing changes.  */
+  *again = FALSE;
+
+  /* We don't have to do anything for a relocatable link, if
+     this section does not have relocs, or if this is not a
+     code section.  */
+  if (link_info->relocatable
+      || (sec->flags & SEC_RELOC) == 0
+      || sec->reloc_count == 0 || (sec->flags & SEC_CODE) == 0)
+    return TRUE;
+
+  symtab_hdr = & elf_tdata (abfd)->symtab_hdr;
+
+  /* Get a copy of the native relocations.  */
+  internal_relocs =
+    _bfd_elf_link_read_relocs (abfd, sec, NULL, NULL, link_info->keep_memory);
+  if (internal_relocs == NULL)
+    goto error_return;
+
+  /* Walk through them looking for relaxing opportunities.  */
+  irelend = internal_relocs + sec->reloc_count;
+  for (irel = internal_relocs; irel < irelend; irel++)
+    {
+      bfd_vma symval;
+
+      /* If this isn't something that can be relaxed, then ignore
+         this reloc.  */
+      if (ELF32_R_TYPE (irel->r_info) != (int) R_MSP430_RL_PCREL)
+       continue;
+
+      /* Get the section contents if we haven't done so already.  */
+      if (contents == NULL)
+       {
+         /* Get cached copy if it exists.  */
+         if (elf_section_data (sec)->this_hdr.contents != NULL)
+           contents = elf_section_data (sec)->this_hdr.contents;
+         else if (! bfd_malloc_and_get_section (abfd, sec, &contents))
+           goto error_return;
+       }
+
+      /* Read this BFD's local symbols if we haven't done so already.  */
+      if (isymbuf == NULL && symtab_hdr->sh_info != 0)
+       {
+         isymbuf = (Elf_Internal_Sym *) symtab_hdr->contents;
+         if (isymbuf == NULL)
+           isymbuf = bfd_elf_get_elf_syms (abfd, symtab_hdr,
+                                           symtab_hdr->sh_info, 0,
+                                           NULL, NULL, NULL);
+         if (isymbuf == NULL)
+           goto error_return;
+       }
+
+      /* Get the value of the symbol referred to by the reloc.  */
+      if (ELF32_R_SYM (irel->r_info) < symtab_hdr->sh_info)
+       {
+         /* A local symbol.  */
+         Elf_Internal_Sym *isym;
+         asection *sym_sec;
+
+         isym = isymbuf + ELF32_R_SYM (irel->r_info);
+         if (isym->st_shndx == SHN_UNDEF)
+           sym_sec = bfd_und_section_ptr;
+         else if (isym->st_shndx == SHN_ABS)
+           sym_sec = bfd_abs_section_ptr;
+         else if (isym->st_shndx == SHN_COMMON)
+           sym_sec = bfd_com_section_ptr;
+         else
+           sym_sec = bfd_section_from_elf_index (abfd, isym->st_shndx);
+         symval = (isym->st_value
+                   + sym_sec->output_section->vma + sym_sec->output_offset);
+       }
+      else
+       {
+         unsigned long indx;
+         struct elf_link_hash_entry *h;
+
+         /* An external symbol.  */
+         indx = ELF32_R_SYM (irel->r_info) - symtab_hdr->sh_info;
+         h = elf_sym_hashes (abfd)[indx];
+         BFD_ASSERT (h != NULL);
+
+         if (h->root.type != bfd_link_hash_defined
+             && h->root.type != bfd_link_hash_defweak)
+           /* This appears to be a reference to an undefined
+              symbol.  Just ignore it--it will be caught by the
+              regular reloc processing.  */
+           continue;
+
+         symval = (h->root.u.def.value
+                   + h->root.u.def.section->output_section->vma
+                   + h->root.u.def.section->output_offset);
+       }
+
+      /* For simplicity of coding, we are going to modify the section
+         contents, the section relocs, and the BFD symbol table.  We
+         must tell the rest of the code not to free up this
+         information.  It would be possible to instead create a table
+         of changes which have to be made, as is done in coff-mips.c;
+         that would be more work, but would require less memory when
+         the linker is run.  */
+
+      /* Try to turn a 16bit pc-relative branch into a 10bit pc-relative
+         branch.  */
+      /* Paranoia? paranoia...  */      
+      if (ELF32_R_TYPE (irel->r_info) == (int) R_MSP430_RL_PCREL)
+       {
+         bfd_vma value = symval;
+
+         /* Deal with pc-relative gunk.  */
+         value -= (sec->output_section->vma + sec->output_offset);
+         value -= irel->r_offset;
+         value += irel->r_addend;
+
+         /* See if the value will fit in 10 bits, note the high value is
+            1016 as the target will be two bytes closer if we are
+            able to relax. */
+         if ((long) value < 1016 && (long) value > -1016)
+           {
+             int code0 = 0, code1 = 0, code2 = 0;
+             int i;
+             struct rcodes_s *rx;
+
+             /* Get the opcode.  */
+             if (irel->r_offset >= 6)
+               code0 = bfd_get_16 (abfd, contents + irel->r_offset - 6);
+
+             if (irel->r_offset >= 4)
+               code1 = bfd_get_16 (abfd, contents + irel->r_offset - 4);
+
+             code2 = bfd_get_16 (abfd, contents + irel->r_offset - 2);
+
+             if (code2 != 0x4010)
+               continue;
+
+             /* Check r4 and r3.  */
+             for (i = NUMB_RELAX_CODES - 1; i >= 0; i--)
+               {
+                 rx = &rcode[i];
+                 if (rx->cdx == 2 && rx->f0 == code0 && rx->f1 == code1)
+                   break;
+                 else if (rx->cdx == 1 && rx->f1 == code1)
+                   break;
+                 else if (rx->cdx == 0)        /* This is an unconditional jump.  */
+                   break;
+               }
+
+             /* Check labels:
+                  .Label0:       ; we do not care about this label
+                     jeq    +6
+                  .Label1:       ; make sure there is no label here
+                     jl     +4
+                  .Label2:       ; make sure there is no label here
+                     br .Label_dst
+
+                So, if there is .Label1 or .Label2 we cannot relax this code.
+                This actually should not happen, cause for relaxable
+                instructions we use RL_PCREL reloc instead of 16_PCREL.
+                Will change this in the future. */
+
+             if (rx->cdx > 0
+                 && msp430_elf_symbol_address_p (abfd, sec, isymbuf,
+                                                 irel->r_offset - 2))
+               continue;
+             if (rx->cdx > 1
+                 && msp430_elf_symbol_address_p (abfd, sec, isymbuf,
+                                                 irel->r_offset - 4))
+               continue;
+
+             /* Note that we've changed the relocs, section contents, etc.  */
+             elf_section_data (sec)->relocs = internal_relocs;
+             elf_section_data (sec)->this_hdr.contents = contents;
+             symtab_hdr->contents = (unsigned char *) isymbuf;
+
+             /* Fix the relocation's type.  */
+             if (rx->labels == 3)      /* Handle special cases.  */
+               irel->r_info = ELF32_R_INFO (ELF32_R_SYM (irel->r_info),
+                                          R_MSP430_2X_PCREL);
+             else
+               irel->r_info = ELF32_R_INFO (ELF32_R_SYM (irel->r_info),
+                                          R_MSP430_10_PCREL);
+
+             /* Fix the opcode right way.  */
+             bfd_put_16 (abfd, rx->t0, contents + irel->r_offset - rx->off);
+             if (rx->t1)
+               bfd_put_16 (abfd, rx->t1,
+                           contents + irel->r_offset - rx->off + 2);
+
+             /* Delete bytes. */
+             if (!msp430_elf_relax_delete_bytes (abfd, sec,
+                                                 irel->r_offset - rx->off +
+                                                 rx->ncl, rx->bs))
+               goto error_return;
+
+             /* Handle unconditional jumps.  */
+             if (rx->cdx == 0)
+               irel->r_offset -= 2;
+
+             /* That will change things, so, we should relax again.
+                Note that this is not required, and it may be slow.  */
+             *again = TRUE;
+           }
+       }
+    }
+
+  if (isymbuf != NULL && symtab_hdr->contents != (unsigned char *) isymbuf)
+    {
+      if (!link_info->keep_memory)
+       free (isymbuf);
+      else
+       {
+         /* Cache the symbols for elf_link_input_bfd.  */
+         symtab_hdr->contents = (unsigned char *) isymbuf;
+       }
+    }
+
+  if (contents != NULL
+      && elf_section_data (sec)->this_hdr.contents != contents)
+    {
+      if (!link_info->keep_memory)
+       free (contents);
+      else
+       {
+         /* Cache the section contents for elf_link_input_bfd.  */
+         elf_section_data (sec)->this_hdr.contents = contents;
+       }
+    }
+
+  if (internal_relocs != NULL
+      && elf_section_data (sec)->relocs != internal_relocs)
+    free (internal_relocs);
+
+  return TRUE;
+
+error_return:
+  if (isymbuf != NULL && symtab_hdr->contents != (unsigned char *) isymbuf)
+    free (isymbuf);
+  if (contents != NULL
+      && elf_section_data (sec)->this_hdr.contents != contents)
+    free (contents);
+  if (internal_relocs != NULL
+      && elf_section_data (sec)->relocs != internal_relocs)
+    free (internal_relocs);
+
+  return FALSE;
+}
+
 
 #define ELF_ARCH               bfd_arch_msp430
 #define ELF_MACHINE_CODE       EM_MSP430
@@ -703,5 +1173,6 @@ elf32_msp430_post_process_headers (abfd, link_info)
 #define elf_backend_final_write_processing   bfd_elf_msp430_final_write_processing
 #define elf_backend_object_p                elf32_msp430_object_p
 #define elf_backend_post_process_headers     elf32_msp430_post_process_headers
+#define bfd_elf32_bfd_relax_section         msp430_elf_relax_section
 
 #include "elf32-target.h"
index de3ead65d06a73dd464b17a0619acfeea8da6f74..4fe194ee8edc98e53e93e1ac24c11cbce4343c8b 100644 (file)
@@ -1637,6 +1637,8 @@ static const char *const bfd_reloc_code_real_names[] = { "@@uninitialized@@",
   "BFD_RELOC_MSP430_16",
   "BFD_RELOC_MSP430_16_PCREL_BYTE",
   "BFD_RELOC_MSP430_16_BYTE",
+  "BFD_RELOC_MSP430_2X_PCREL",
+  "BFD_RELOC_MSP430_RL_PCREL",
   "BFD_RELOC_IQ2000_OFFSET_16",
   "BFD_RELOC_IQ2000_OFFSET_21",
   "BFD_RELOC_IQ2000_UHI16",
index 48166db51ec55ee8bd09895d72521f7a5861461e..9c1ed33f1ab0e081b7f3c447678786424b2f5a33 100644 (file)
@@ -4095,6 +4095,10 @@ ENUMX
   BFD_RELOC_MSP430_16_PCREL_BYTE
 ENUMX
   BFD_RELOC_MSP430_16_BYTE
+ENUMX
+  BFD_RELOC_MSP430_2X_PCREL
+ENUMX
+  BFD_RELOC_MSP430_RL_PCREL
 ENUMDOC
   msp430 specific relocation codes
 
index 4eec4a80288febb7865604354564aca8a7ff5c25..2c96af949835f0ee960f787421bc844a764c773d 100644 (file)
@@ -1,3 +1,20 @@
+2004-08-25  Dmitry Diky  <diwil@spec.ru>
+
+       * config/tc-msp430.c:   Clean-up the code.
+       (md_relax_table): New relax table.
+       (mcu_types): Sort MCU types.
+       (md_pseudo_table): Add .profiler pseudo handler.
+       (pow2value): New function.
+       (msp430_profiler): New function.
+       (msp430_operands): Add new insns handlers.
+       (msp430_srcoperand): Add register operand handler, allow complex
+       expressions.
+       (md_estimate_size_before_relax): Rewritten.
+       (md_convert_frag): Rewritten.
+       (msp430_relax_frag): New function.
+       * config/tc-msp430.h (md_relax_frag): define macro
+       * doc/c-msp430.texi: Update information.
+
 2004-08-24  Nick Clifton  <nickc@redhat.com>
 
        * as.c (std_shortopts): Allow -g to take an optional argument.
index 2c3d81efbabe9fceb30309ff7549ad53dc7788df..03346f6075b26ae7376db214e998baa1b4a8d4a2 100644 (file)
@@ -42,30 +42,67 @@ extern LITTLENUM_TYPE generic_bignum[];
 
 static struct hash_control *msp430_hash;
 
-static unsigned int msp430_operands
-  PARAMS ((struct msp430_opcode_s *, char *));
-static int msp430_srcoperand
-  PARAMS ((struct msp430_operand_s *, char *, int, int *));
-static int msp430_dstoperand
-  PARAMS ((struct msp430_operand_s *, char *, int));
-static char *parse_exp
-  PARAMS ((char *, expressionS *));
-static inline char *skip_space
-  PARAMS ((char *));
-static int check_reg
-  PARAMS ((char *));
-static void msp430_set_arch
-  PARAMS ((int));
-static void show_mcu_list
-  PARAMS ((FILE *));
-static void del_spaces
-  PARAMS ((char *));
+/* Relaxations.  */
+#define STATE_UNCOND_BRANCH    1       /* jump */
+#define STATE_NOOV_BRANCH      3       /* bltn */
+#define STATE_SIMPLE_BRANCH    2       /* bne, beq, etc... */
+#define STATE_EMUL_BRANCH      4
+
+#define CNRL   2
+#define CUBL   4
+#define CNOL   8
+#define CSBL   6
+#define CEBL   4
+
+/* Length.  */
+#define STATE_BITS10   1       /* wild guess. short jump */
+#define STATE_WORD     2       /* 2 bytes pc rel. addr. more */
+#define STATE_UNDEF    3       /* cannot handle this yet. convert to word mode */
+
+#define ENCODE_RELAX(what,length) (((what) << 2) + (length))
+#define RELAX_STATE(s)            ((s) & 3)
+#define RELAX_LEN(s)             ((s) >> 2)
+#define RELAX_NEXT(a,b)                  ENCODE_RELAX (a, b + 1)
+
+relax_typeS md_relax_table[] =
+{
+  /* Unused.  */
+  {1, 1, 0, 0},
+  {1, 1, 0, 0},
+  {1, 1, 0, 0},
+  {1, 1, 0, 0},
+
+  /* Unconditional jump.  */
+  {1, 1, 8, 5},
+  {1024, -1024, CNRL, RELAX_NEXT (STATE_UNCOND_BRANCH, STATE_BITS10)}, /* state 10 bits displ */
+  {0, 0, CUBL, RELAX_NEXT (STATE_UNCOND_BRANCH, STATE_WORD)},          /* state word */
+  {1, 1, CUBL, 0},                                                     /* state undef */
+
+  /* Simple branches.  */
+  {0, 0, 8, 9},
+  {1024, -1024, CNRL, RELAX_NEXT (STATE_SIMPLE_BRANCH, STATE_BITS10)}, /* state 10 bits displ */
+  {0, 0, CSBL, RELAX_NEXT (STATE_SIMPLE_BRANCH, STATE_WORD)},          /* state word */
+  {1, 1, CSBL, 0},
+
+  /* blt no overflow branch.  */
+  {1, 1, 8, 13},
+  {1024, -1024, CNRL, RELAX_NEXT (STATE_NOOV_BRANCH, STATE_BITS10)},   /* state 10 bits displ */
+  {0, 0, CNOL, RELAX_NEXT (STATE_NOOV_BRANCH, STATE_WORD)},            /* state word */
+  {1, 1, CNOL, 0},
+
+  /* Emulated branches.  */
+  {1, 1, 8, 17},
+  {1020, -1020, CEBL, RELAX_NEXT (STATE_EMUL_BRANCH, STATE_BITS10)},   /* state 10 bits displ */
+  {0, 0, CNOL, RELAX_NEXT (STATE_EMUL_BRANCH, STATE_WORD)},            /* state word */
+  {1, 1, CNOL, 0}
+};
+
 
 #define MAX_OP_LEN     256
 
 struct mcu_type_s
 {
-  char *name;
+  char * name;
   int isa;
   int mach;
 };
@@ -90,53 +127,53 @@ struct mcu_type_s
 
 static struct mcu_type_s mcu_types[] =
 {
-  {"msp1",       MSP430_ISA_11, bfd_mach_msp11},
-  {"msp2",       MSP430_ISA_14, bfd_mach_msp14},
-  {"msp430x110", MSP430_ISA_11, bfd_mach_msp11},
-  {"msp430x112", MSP430_ISA_11, bfd_mach_msp11},
-  {"msp430x1101",MSP430_ISA_110, bfd_mach_msp110},
-  {"msp430x1111",MSP430_ISA_110, bfd_mach_msp110},
-  {"msp430x1121",MSP430_ISA_110, bfd_mach_msp110},
-  {"msp430x1122",MSP430_ISA_11, bfd_mach_msp110},
-  {"msp430x1132",MSP430_ISA_11, bfd_mach_msp110},
-
-  {"msp430x122", MSP430_ISA_12, bfd_mach_msp12},
-  {"msp430x123", MSP430_ISA_12, bfd_mach_msp12},
-  {"msp430x1222",MSP430_ISA_12, bfd_mach_msp12},
-  {"msp430x1232",MSP430_ISA_12, bfd_mach_msp12},
-
-  {"msp430x133", MSP430_ISA_13, bfd_mach_msp13},
-  {"msp430x135", MSP430_ISA_13, bfd_mach_msp13},
-  {"msp430x1331",MSP430_ISA_13, bfd_mach_msp13},
-  {"msp430x1351",MSP430_ISA_13, bfd_mach_msp13},
-  {"msp430x147", MSP430_ISA_14, bfd_mach_msp14},
-  {"msp430x148", MSP430_ISA_14, bfd_mach_msp14},
-  {"msp430x149", MSP430_ISA_14, bfd_mach_msp14},
-
-  {"msp430x155", MSP430_ISA_15, bfd_mach_msp15},
-  {"msp430x156", MSP430_ISA_15, bfd_mach_msp15},
-  {"msp430x157", MSP430_ISA_15, bfd_mach_msp15},
-  {"msp430x167", MSP430_ISA_16, bfd_mach_msp16},
-  {"msp430x168", MSP430_ISA_16, bfd_mach_msp16},
-  {"msp430x169", MSP430_ISA_16, bfd_mach_msp16},
+  {"msp1",        MSP430_ISA_11, bfd_mach_msp11},
+  {"msp2",        MSP430_ISA_14, bfd_mach_msp14},
+  {"msp430x110",  MSP430_ISA_11, bfd_mach_msp11},
+  {"msp430x112",  MSP430_ISA_11, bfd_mach_msp11},
+  {"msp430x1101", MSP430_ISA_110, bfd_mach_msp110},
+  {"msp430x1111", MSP430_ISA_110, bfd_mach_msp110},
+  {"msp430x1121", MSP430_ISA_110, bfd_mach_msp110},
+  {"msp430x1122", MSP430_ISA_11, bfd_mach_msp110},
+  {"msp430x1132", MSP430_ISA_11, bfd_mach_msp110},
+
+  {"msp430x122",  MSP430_ISA_12, bfd_mach_msp12},
+  {"msp430x123",  MSP430_ISA_12, bfd_mach_msp12},
+  {"msp430x1222", MSP430_ISA_12, bfd_mach_msp12},
+  {"msp430x1232", MSP430_ISA_12, bfd_mach_msp12},
+
+  {"msp430x133",  MSP430_ISA_13, bfd_mach_msp13},
+  {"msp430x135",  MSP430_ISA_13, bfd_mach_msp13},
+  {"msp430x1331", MSP430_ISA_13, bfd_mach_msp13},
+  {"msp430x1351", MSP430_ISA_13, bfd_mach_msp13},
+  {"msp430x147",  MSP430_ISA_14, bfd_mach_msp14},
+  {"msp430x148",  MSP430_ISA_14, bfd_mach_msp14},
+  {"msp430x149",  MSP430_ISA_14, bfd_mach_msp14},
+
+  {"msp430x155",  MSP430_ISA_15, bfd_mach_msp15},
+  {"msp430x156",  MSP430_ISA_15, bfd_mach_msp15},
+  {"msp430x157",  MSP430_ISA_15, bfd_mach_msp15},
+  {"msp430x167",  MSP430_ISA_16, bfd_mach_msp16},
+  {"msp430x168",  MSP430_ISA_16, bfd_mach_msp16},
+  {"msp430x169",  MSP430_ISA_16, bfd_mach_msp16},
   {"msp430x1610", MSP430_ISA_16, bfd_mach_msp16},
   {"msp430x1611", MSP430_ISA_16, bfd_mach_msp16},
   {"msp430x1612", MSP430_ISA_16, bfd_mach_msp16},
 
-  {"msp430x311", MSP430_ISA_31, bfd_mach_msp31},
-  {"msp430x312", MSP430_ISA_31, bfd_mach_msp31},
-  {"msp430x313", MSP430_ISA_31, bfd_mach_msp31},
-  {"msp430x314", MSP430_ISA_31, bfd_mach_msp31},
-  {"msp430x315", MSP430_ISA_31, bfd_mach_msp31},
-  {"msp430x323", MSP430_ISA_32, bfd_mach_msp32},
-  {"msp430x325", MSP430_ISA_32, bfd_mach_msp32},
-  {"msp430x336", MSP430_ISA_33, bfd_mach_msp33},
-  {"msp430x337", MSP430_ISA_33, bfd_mach_msp33},
-
-  {"msp430x412", MSP430_ISA_41, bfd_mach_msp41},
-  {"msp430x413", MSP430_ISA_41, bfd_mach_msp41},
-  {"msp430x415", MSP430_ISA_41, bfd_mach_msp41},
-  {"msp430x417", MSP430_ISA_41, bfd_mach_msp41},
+  {"msp430x311",  MSP430_ISA_31, bfd_mach_msp31},
+  {"msp430x312",  MSP430_ISA_31, bfd_mach_msp31},
+  {"msp430x313",  MSP430_ISA_31, bfd_mach_msp31},
+  {"msp430x314",  MSP430_ISA_31, bfd_mach_msp31},
+  {"msp430x315",  MSP430_ISA_31, bfd_mach_msp31},
+  {"msp430x323",  MSP430_ISA_32, bfd_mach_msp32},
+  {"msp430x325",  MSP430_ISA_32, bfd_mach_msp32},
+  {"msp430x336",  MSP430_ISA_33, bfd_mach_msp33},
+  {"msp430x337",  MSP430_ISA_33, bfd_mach_msp33},
+
+  {"msp430x412",  MSP430_ISA_41, bfd_mach_msp41},
+  {"msp430x413",  MSP430_ISA_41, bfd_mach_msp41},
+  {"msp430x415",  MSP430_ISA_41, bfd_mach_msp41},
+  {"msp430x417",  MSP430_ISA_41, bfd_mach_msp41},
 
   {"msp430xE423", MSP430_ISA_42, bfd_mach_msp42},
   {"msp430xE425", MSP430_ISA_42, bfd_mach_msp42},
@@ -150,12 +187,12 @@ static struct mcu_type_s mcu_types[] =
   {"msp430xG438", MSP430_ISA_43, bfd_mach_msp43},
   {"msp430xG439", MSP430_ISA_43, bfd_mach_msp43},
 
-  {"msp430x435", MSP430_ISA_43, bfd_mach_msp43},
-  {"msp430x436", MSP430_ISA_43, bfd_mach_msp43},
-  {"msp430x437", MSP430_ISA_43, bfd_mach_msp43},
-  {"msp430x447", MSP430_ISA_44, bfd_mach_msp44},
-  {"msp430x448", MSP430_ISA_44, bfd_mach_msp44},
-  {"msp430x449", MSP430_ISA_44, bfd_mach_msp44},
+  {"msp430x435",  MSP430_ISA_43, bfd_mach_msp43},
+  {"msp430x436",  MSP430_ISA_43, bfd_mach_msp43},
+  {"msp430x437",  MSP430_ISA_43, bfd_mach_msp43},
+  {"msp430x447",  MSP430_ISA_44, bfd_mach_msp44},
+  {"msp430x448",  MSP430_ISA_44, bfd_mach_msp44},
+  {"msp430x449",  MSP430_ISA_44, bfd_mach_msp44},
 
   {NULL, 0, 0}
 };
@@ -164,72 +201,346 @@ static struct mcu_type_s mcu_types[] =
 static struct mcu_type_s default_mcu =
     { "msp430x11", MSP430_ISA_11, bfd_mach_msp11 };
 
-static struct mcu_type_s *msp430_mcu = &default_mcu;
+static struct mcu_type_s * msp430_mcu = & default_mcu;
+
+/* Profiling capability:
+   It is a performance hit to use gcc's profiling approach for this tiny target.
+   Even more -- jtag hardware facility does not perform any profiling functions.
+   However we've got gdb's built-in simulator where we can do anything.
+   Therefore my suggestion is:
+
+   We define new section ".profiler" which holds all profiling information.
+   We define new pseudo operation .profiler which will instruct assembler to
+   add new profile entry to the object file. Profile should take place at the
+   present address.
+
+   Pseudo-op format:
+
+      .profiler flags,function_to_profile [, cycle_corrector, extra]
+
+   where 'flags' is a combination of the following chars:
+           s - function Start
+           x - function eXit
+           i - function is in Init section
+           f - function is in Fini section
+           l - Library call
+           c - libC standard call
+           d - stack value Demand (saved at run-time in simulator)
+           I - Interrupt service routine
+           P - Prologue start
+           p - Prologue end
+           E - Epilogue start
+           e - Epilogue end
+           j - long Jump/ sjlj unwind
+           a - an Arbitrary code fragment
+           t - exTra parameter saved (constant value like frame size)
+         '""' optional: "sil" == sil
+
+      function_to_profile - function's address
+      cycle_corrector     - a value which should be added to the cycle
+                             counter, zero if omitted
+      extra - some extra parameter, zero if omitted.
+
+      For example:
+      ------------------------------
+       .global fxx
+       .type fxx,@function
+      fxx:
+      .LFrameOffset_fxx=0x08
+      .profiler "scdP", fxx    ; function entry.
+                               ; we also demand stack value to be displayed
+       push r11
+       push r10
+       push r9
+       push r8
+      .profiler "cdp",fxx,0, .LFrameOffset_fxx ; check stack value at this point
+                                               ; (this is a prologue end)
+                                               ; note, that spare var filled with the farme size
+       mov r15,r8
+       ....
+      .profiler cdE,fxx                ; check stack
+       pop r8
+       pop r9
+       pop r10
+       pop r11
+      .profiler xcde,fxx,3     ; exit adds 3 to the cycle counter
+      ret                      ; cause 'ret' insn takes 3 cycles
+      -------------------------------
+
+      This profiling approach does not produce any overhead and
+      absolutely harmless.
+      So, even profiled code can be uploaded to the MCU.  */
+#define MSP430_PROFILER_FLAG_ENTRY     1       /* s */
+#define MSP430_PROFILER_FLAG_EXIT      2       /* x */
+#define MSP430_PROFILER_FLAG_INITSECT  4       /* i */
+#define MSP430_PROFILER_FLAG_FINISECT  8       /* f */
+#define MSP430_PROFILER_FLAG_LIBCALL   0x10    /* l */
+#define MSP430_PROFILER_FLAG_STDCALL   0x20    /* c */
+#define MSP430_PROFILER_FLAG_STACKDMD  0x40    /* d */
+#define MSP430_PROFILER_FLAG_ISR       0x80    /* I */
+#define MSP430_PROFILER_FLAG_PROLSTART 0x100   /* P */
+#define MSP430_PROFILER_FLAG_PROLEND   0x200   /* p */
+#define MSP430_PROFILER_FLAG_EPISTART  0x400   /* E */
+#define MSP430_PROFILER_FLAG_EPIEND    0x800   /* e */
+#define MSP430_PROFILER_FLAG_JUMP      0x1000  /* j */
+#define MSP430_PROFILER_FLAG_FRAGMENT  0x2000  /* a */
+#define MSP430_PROFILER_FLAG_EXTRA     0x4000  /* t */
+#define MSP430_PROFILER_FLAG_notyet    0x8000  /* ? */
 
-const pseudo_typeS md_pseudo_table[] =
+static int
+pow2value (int y)
 {
-  {"arch", msp430_set_arch, 0},
-  {NULL, NULL, 0}
-};
+  int n = 0;
+  unsigned int x;
 
-#define OPTION_MMCU 'm'
+  x = y;
 
-const char *md_shortopts = "m:";
+  if (!x)
+    return 1;
 
-struct option md_longopts[] =
+  for (; x; x = x >> 1)
+    if (x & 1)
+      n++;
+
+  return n == 1;
+}
+
+/* Parse ordinary expression.  */
+
+static char *
+parse_exp (char * s, expressionS * op)
 {
-  {"mmcu", required_argument, NULL, OPTION_MMCU},
-  {NULL, no_argument, NULL, 0}
-};
+  input_line_pointer = s;
+  expression (op);
+  if (op->X_op == O_absent)
+    as_bad (_("missing operand"));
+  return input_line_pointer;
+}
 
-size_t md_longopts_size = sizeof (md_longopts);
+
+/* Delete spaces from s: X ( r 1  2)  => X(r12).  */
 
 static void
-show_mcu_list (stream)
-     FILE *stream;
+del_spaces (char * s)
 {
-  int i;
+  while (*s)
+    {
+      if (ISSPACE (*s))
+       {
+         char *m = s + 1;
 
-  fprintf (stream, _("Known MCU names:\n"));
+         while (ISSPACE (*m) && *m)
+           m++;
+         memmove (s, m, strlen (m) + 1);
+       }
+      else
+       s++;
+    }
+}
 
-  for (i = 0; mcu_types[i].name; i++)
-    fprintf (stream, _("\t %s\n"), mcu_types[i].name);
+static inline char *
+skip_space (char * s)
+{
+  while (ISSPACE (*s))
+    ++s;
+  return s;
+}
 
-  fprintf (stream, "\n");
+/* Extract one word from FROM and copy it to TO. Delimeters are ",;\n"  */
+
+static char *
+extract_operand (char * from, char * to, int limit)
+{
+  int size = 0;
+
+  /* Drop leading whitespace.  */
+  from = skip_space (from);
+
+  while (size < limit && *from)
+    {
+      *(to + size) = *from;
+      if (*from == ',' || *from == ';' || *from == '\n')
+       break;
+      from++;
+      size++;
+    }
+
+  *(to + size) = 0;
+  del_spaces (to);
+
+  from++;
+
+  return from;
 }
 
-void
-md_show_usage (FILE *stream)
+static void
+msp430_profiler (int dummy ATTRIBUTE_UNUSED)
 {
-  fprintf (stream,
-          _("MSP430 options:\n"
-            "  -mmcu=[msp430-name] select microcontroller type\n"
-            "                  msp430x110  msp430x112\n"
-            "                  msp430x1101 msp430x1111\n"
-            "                  msp430x1121 msp430x1122 msp430x1132\n"
-            "                  msp430x122  msp430x123\n"
-            "                  msp430x1222 msp430x1232\n"
-            "                  msp430x133  msp430x135\n"
-            "                  msp430x1331 msp430x1351\n"
-            "                  msp430x147  msp430x148  msp430x149\n"
-            "                  msp430x155  msp430x156  msp430x157\n"
-            "                  msp430x167  msp430x168  msp430x169\n"
-            "                  msp430x1610 msp430x1611 msp430x1612\n"
-            "                  msp430x311  msp430x312  msp430x313  msp430x314  msp430x315\n"
-            "                  msp430x323  msp430x325\n"
-            "                  msp430x336  msp430x337\n"
-            "                  msp430x412  msp430x413  msp430x415  msp430x417\n"
-            "                  msp430xE423 msp430xE425 msp430E427\n"
-            "                  msp430xW423 msp430xW425 msp430W427\n"
-            "                  msp430xG437 msp430xG438 msp430G439\n"
-            "                  msp430x435  msp430x436  msp430x437\n"
-            "                  msp430x447  msp430x448  msp430x449\n"));
+  char   buffer[1024];
+  char   f[32];
+  char * str = buffer;
+  char * flags = f;
+  int    p_flags = 0;
+  char * halt;
+  int    ops = 0;
+  int    left;
+  char * s;
+  segT   seg;
+  int    subseg;
+  char * end = 0;
+  expressionS exp;
+  expressionS exp1;
+
+  s = input_line_pointer;
+  end = input_line_pointer;
+
+  while (*end && *end != '\n')
+    end++;
+
+  while (*s && *s != '\n')
+    {
+      if (*s == ',')
+       ops++;
+      s++;
+    }
 
-  show_mcu_list (stream);
+  left = 3 - ops;
+
+  if (ops < 1)
+    {
+      as_bad (_(".profiler pseudo requires at least two operands."));
+      input_line_pointer = end;
+      return;
+    }
+
+  input_line_pointer = extract_operand (input_line_pointer, flags, 32);
+
+  while (*flags)
+    {
+      switch (*flags)
+       {
+       case '"':
+         break;
+       case 'a':
+         p_flags |= MSP430_PROFILER_FLAG_FRAGMENT;
+         break;
+       case 'j':
+         p_flags |= MSP430_PROFILER_FLAG_JUMP;
+         break;
+       case 'P':
+         p_flags |= MSP430_PROFILER_FLAG_PROLSTART;
+         break;
+       case 'p':
+         p_flags |= MSP430_PROFILER_FLAG_PROLEND;
+         break;
+       case 'E':
+         p_flags |= MSP430_PROFILER_FLAG_EPISTART;
+         break;
+       case 'e':
+         p_flags |= MSP430_PROFILER_FLAG_EPIEND;
+         break;
+       case 's':
+         p_flags |= MSP430_PROFILER_FLAG_ENTRY;
+         break;
+       case 'x':
+         p_flags |= MSP430_PROFILER_FLAG_EXIT;
+         break;
+       case 'i':
+         p_flags |= MSP430_PROFILER_FLAG_INITSECT;
+         break;
+       case 'f':
+         p_flags |= MSP430_PROFILER_FLAG_FINISECT;
+         break;
+       case 'l':
+         p_flags |= MSP430_PROFILER_FLAG_LIBCALL;
+         break;
+       case 'c':
+         p_flags |= MSP430_PROFILER_FLAG_STDCALL;
+         break;
+       case 'd':
+         p_flags |= MSP430_PROFILER_FLAG_STACKDMD;
+         break;
+       case 'I':
+         p_flags |= MSP430_PROFILER_FLAG_ISR;
+         break;
+       case 't':
+         p_flags |= MSP430_PROFILER_FLAG_EXTRA;
+         break;
+       default:
+         as_warn (_("unknown profiling flag - ignored."));
+         break;
+       }
+      flags++;
+    }
+
+  if (p_flags
+      && (   ! pow2value (p_flags & (  MSP430_PROFILER_FLAG_ENTRY
+                                    | MSP430_PROFILER_FLAG_EXIT))
+         || ! pow2value (p_flags & (  MSP430_PROFILER_FLAG_PROLSTART
+                                    | MSP430_PROFILER_FLAG_PROLEND
+                                    | MSP430_PROFILER_FLAG_EPISTART
+                                    | MSP430_PROFILER_FLAG_EPIEND))
+         || ! pow2value (p_flags & (  MSP430_PROFILER_FLAG_INITSECT
+                                    | MSP430_PROFILER_FLAG_FINISECT))))
+    {
+      as_bad (_("ambigious flags combination - '.profiler' directive ignored."));
+      input_line_pointer = end;
+      return;
+    }
+
+  /* Generate temp symbol which denotes current location.  */
+  if (now_seg == absolute_section)     /* Paranoja ?  */
+    {
+      exp1.X_op = O_constant;
+      exp1.X_add_number = abs_section_offset;
+      as_warn (_("profiling in absolute section? Hm..."));
+    }
+  else
+    {
+      exp1.X_op = O_symbol;
+      exp1.X_add_symbol = symbol_temp_new_now ();
+      exp1.X_add_number = 0;
+    }
+
+  /* Generate a symbol which holds flags value.  */
+  exp.X_op = O_constant;
+  exp.X_add_number = p_flags;
+
+  /* Save current section.  */
+  seg = now_seg;
+  subseg = now_subseg;
+
+  /* Now go to .profiler section.  */
+  obj_elf_change_section (".profiler", SHT_PROGBITS, 0, 0, 0, 0, 0);
+
+  /* Save flags.  */
+  emit_expr (& exp, 2);
+
+  /* Save label value.  */
+  emit_expr (& exp1, 2);
+
+  while (ops--)
+    {
+      /* Now get profiling info.  */
+      halt = extract_operand (input_line_pointer, str, 1024);
+      /* Process like ".word xxx" directive.  */
+      parse_exp (str, & exp);
+      emit_expr (& exp, 2);
+      input_line_pointer = halt;
+    }
+
+  /* Fill the rest with zeros.  */
+  exp.X_op = O_constant;
+  exp.X_add_number = 0;
+  while (left--)
+    emit_expr (& exp, 2);
+
+  /* Return to current section.  */
+  subseg_set (seg, subseg);
 }
 
 static char *
-extract_word (char *from, char *to, int limit)
+extract_word (char * from, char * to, int limit)
 {
   char *op_start;
   char *op_end;
@@ -251,9 +562,10 @@ extract_word (char *from, char *to, int limit)
   return op_end;
 }
 
+#define OPTION_MMCU 'm'
+
 static void
-msp430_set_arch (dummy)
-     int dummy ATTRIBUTE_UNUSED;
+msp430_set_arch (int dummy ATTRIBUTE_UNUSED)
 {
   char *str = (char *) alloca (32);    /* 32 for good measure.  */
 
@@ -263,10 +575,21 @@ msp430_set_arch (dummy)
   bfd_set_arch_mach (stdoutput, TARGET_ARCH, msp430_mcu->mach);
 }
 
+static void
+show_mcu_list (FILE * stream)
+{
+  int i;
+
+  fprintf (stream, _("Known MCU names:\n"));
+
+  for (i = 0; mcu_types[i].name; i++)
+    fprintf (stream, _("\t %s\n"), mcu_types[i].name);
+
+  fprintf (stream, "\n");
+}
+
 int
-md_parse_option (c, arg)
-     int c;
-     char *arg;
+md_parse_option (int c, char * arg)
 {
   int i;
 
@@ -294,72 +617,62 @@ md_parse_option (c, arg)
   return 0;
 }
 
-symbolS *
-md_undefined_symbol (name)
-     char *name ATTRIBUTE_UNUSED;
-{
-  return 0;
-}
 
-static inline char *
-skip_space (s)
-     char *s;
+const pseudo_typeS md_pseudo_table[] =
 {
-  while (ISSPACE (*s))
-    ++s;
-  return s;
-}
+  {"arch", msp430_set_arch, 0},
+  {"profiler", msp430_profiler, 0},
+  {NULL, NULL, 0}
+};
 
-/* Delete spaces from s: X ( r 1  2)  => X(r12).  */
+const char *md_shortopts = "m:";
 
-static void
-del_spaces (s)
-     char *s;
+struct option md_longopts[] =
 {
-  while (*s)
-    {
-      if (ISSPACE (*s))
-       {
-         char *m = s + 1;
-
-         while (ISSPACE (*m) && *m)
-           m++;
-         memmove (s, m, strlen (m) + 1);
-       }
-      else
-       s++;
-    }
-}
+  {"mmcu", required_argument, NULL, OPTION_MMCU},
+  {NULL, no_argument, NULL, 0}
+};
 
-/* Extract one word from FROM and copy it to TO. Delimeters are ",;\n"  */
+size_t md_longopts_size = sizeof (md_longopts);
 
-static char *
-extract_operand (char *from, char *to, int limit)
+void
+md_show_usage (FILE * stream)
 {
-  int size = 0;
-
-  /* Drop leading whitespace.  */
-  from = skip_space (from);
-
-  while (size < limit && *from)
-    {
-      *(to + size) = *from;
-      if (*from == ',' || *from == ';' || *from == '\n')
-       break;
-      from++;
-      size++;
-    }
-
-  *(to + size) = 0;
-  del_spaces (to);
+  fprintf (stream,
+          _("MSP430 options:\n"
+            "  -mmcu=[msp430-name] select microcontroller type\n"
+            "                  msp430x110  msp430x112\n"
+            "                  msp430x1101 msp430x1111\n"
+            "                  msp430x1121 msp430x1122 msp430x1132\n"
+            "                  msp430x122  msp430x123\n"
+            "                  msp430x1222 msp430x1232\n"
+            "                  msp430x133  msp430x135\n"
+            "                  msp430x1331 msp430x1351\n"
+            "                  msp430x147  msp430x148  msp430x149\n"
+            "                  msp430x155  msp430x156  msp430x157\n"
+            "                  msp430x167  msp430x168  msp430x169\n"
+            "                  msp430x1610 msp430x1611 msp430x1612\n"
+            "                  msp430x311  msp430x312  msp430x313  msp430x314  msp430x315\n"
+            "                  msp430x323  msp430x325\n"
+            "                  msp430x336  msp430x337\n"
+            "                  msp430x412  msp430x413  msp430x415  msp430x417\n"
+            "                  msp430xE423 msp430xE425 msp430E427\n"
+            "                  msp430xW423 msp430xW425 msp430W427\n"
+            "                  msp430xG437 msp430xG438 msp430G439\n"
+            "                  msp430x435  msp430x436  msp430x437\n"
+            "                  msp430x447  msp430x448  msp430x449\n"));
 
-  from++;
+  show_mcu_list (stream);
+}
 
-  return from;
+symbolS *
+md_undefined_symbol (char * name ATTRIBUTE_UNUSED)
+{
+  return 0;
 }
 
 static char *
-extract_cmd (char *from, char *to, int limit)
+extract_cmd (char * from, char * to, int limit)
 {
   int size = 0;
 
@@ -381,10 +694,7 @@ extract_cmd (char *from, char *to, int limit)
    returned, or NULL on OK.  */
 
 char *
-md_atof (type, litP, sizeP)
-     int type;
-     char *litP;
-     int *sizeP;
+md_atof (int type, char * litP, int * sizeP)
 {
   int prec;
   LITTLENUM_TYPE words[4];
@@ -421,18 +731,9 @@ md_atof (type, litP, sizeP)
 }
 
 void
-md_convert_frag (abfd, sec, fragP)
-     bfd *abfd ATTRIBUTE_UNUSED;
-     asection *sec ATTRIBUTE_UNUSED;
-     fragS *fragP ATTRIBUTE_UNUSED;
-{
-  abort ();
-}
-
-void
-md_begin ()
+md_begin (void)
 {
-  struct msp430_opcode_s *opcode;
+  struct msp430_opcode_s * opcode;
   msp430_hash = hash_new ();
 
   for (opcode = msp430_opcodes; opcode->name; opcode++)
@@ -441,391 +742,506 @@ md_begin ()
   bfd_set_arch_mach (stdoutput, TARGET_ARCH, msp430_mcu->mach);
 }
 
-void
-md_assemble (str)
-     char *str;
+static int
+check_reg (char * t)
 {
-  struct msp430_opcode_s *opcode;
-  char cmd[32];
-  unsigned int i = 0;
-
-  str = skip_space (str);      /* Skip leading spaces.  */
-  str = extract_cmd (str, cmd, sizeof (cmd));
+  /* If this is a reg numb, str 't' must be a number from 0 - 15.  */
 
-  while (cmd[i] && i < sizeof (cmd))
-    {
-      char a = TOLOWER (cmd[i]);
-      cmd[i] = a;
-      i++;
-    }
+  if (strlen (t) > 2 && *(t + 2) != '+')
+    return 1;
 
-  if (!cmd[0])
+  while (*t)
     {
-      as_bad (_("can't find opcode "));
-      return;
+      if ((*t < '0' || *t > '9') && *t != '+')
+       break;
+      t++;
     }
 
-  opcode = (struct msp430_opcode_s *) hash_find (msp430_hash, cmd);
-
-  if (opcode == NULL)
-    {
-      as_bad (_("unknown opcode `%s'"), cmd);
-      return;
-    }
+  if (*t)
+    return 1;
 
-  {
-    char *__t = input_line_pointer;
-    msp430_operands (opcode, str);
-    input_line_pointer = __t;
-  }
+  return 0;
 }
 
-/* Parse instruction operands.
-   Return binary opcode.  */
 
-static unsigned int
-msp430_operands (opcode, line)
-     struct msp430_opcode_s *opcode;
-     char *line;
+static int
+msp430_srcoperand (struct msp430_operand_s * op,
+                  char * l, int bin, int * imm_op)
 {
-  int bin = opcode->bin_opcode;        /* opcode mask.  */
-  int __is;
-  char l1[MAX_OP_LEN], l2[MAX_OP_LEN];
-  char *frag;
-  int where;
-  struct msp430_operand_s op1, op2;
-  int res = 0;
-  static short ZEROS = 0;
-  int byte_op, imm_op;
-
-  /* opcode is the one from opcodes table
-     line contains something like
-     [.w] @r2+, 5(R1)
-     or
-     .b @r2+, 5(R1).  */
+  char *__tl = l;
 
-  /* Check if byte or word operation.  */
-  if (*line == '.' && TOLOWER (*(line + 1)) == 'b')
+  /* Check if an immediate #VALUE.  The hash sign should be only at the beginning!  */
+  if (*l == '#')
     {
-      bin |= BYTE_OPERATION;
-      byte_op = 1;
-    }
-  else
-    byte_op = 0;
+      char *h = l;
+      int vshift = -1;
+      int rval = 0;
 
-  /* skip .[bwBW].  */
-  while (! ISSPACE (*line) && *line)
-    line++;
+      /* Check if there is:
+        llo(x) - least significant 16 bits, x &= 0xffff
+        lhi(x) - x = (x >> 16) & 0xffff,
+        hlo(x) - x = (x >> 32) & 0xffff,
+        hhi(x) - x = (x >> 48) & 0xffff
+        The value _MUST_ be constant expression: #hlo(1231231231).  */
 
-  if (opcode->insn_opnumb && (!*line || *line == '\n'))
-    {
-      as_bad (_("instruction %s requires %d operand(s)"),
-             opcode->name, opcode->insn_opnumb);
-      return 0;
-    }
+      *imm_op = 1;
 
-  memset (l1, 0, sizeof (l1));
-  memset (l2, 0, sizeof (l2));
-  memset (&op1, 0, sizeof (op1));
-  memset (&op2, 0, sizeof (op2));
+      if (strncasecmp (h, "#llo(", 5) == 0)
+       {
+         vshift = 0;
+         rval = 3;
+       }
+      else if (strncasecmp (h, "#lhi(", 5) == 0)
+       {
+         vshift = 1;
+         rval = 3;
+       }
+      else if (strncasecmp (h, "#hlo(", 5) == 0)
+       {
+         vshift = 2;
+         rval = 3;
+       }
+      else if (strncasecmp (h, "#hhi(", 5) == 0)
+       {
+         vshift = 3;
+         rval = 3;
+       }
+      else if (strncasecmp (h, "#lo(", 4) == 0)
+       {
+         vshift = 0;
+         rval = 2;
+       }
+      else if (strncasecmp (h, "#hi(", 4) == 0)
+       {
+         vshift = 1;
+         rval = 2;
+       }
 
-  imm_op = 0;
+      op->reg = 0;             /* Reg PC.  */
+      op->am = 3;
+      op->ol = 1;              /* Immediate  will follow an instruction.  */
+      __tl = h + 1 + rval;
+      op->mode = OP_EXP;
 
-  switch (opcode->fmt)
-    {
-    case 0:                    /* Emulated.  */
-      switch (opcode->insn_opnumb)
+      parse_exp (__tl, &(op->exp));
+      if (op->exp.X_op == O_constant)
        {
-       case 0:
-         /* Set/clear bits instructions.  */
-         __is = 2;
-         frag = frag_more (__is);
-         bfd_putl16 ((bfd_vma) bin, frag);
-         break;
-       case 1:
-         /* Something which works with destination operand.  */
-         line = extract_operand (line, l1, sizeof (l1));
-         res = msp430_dstoperand (&op1, l1, opcode->bin_opcode);
-         if (res)
-           break;
-
-         bin |= (op1.reg | (op1.am << 7));
-         __is = 1 + op1.ol;
-         frag = frag_more (2 * __is);
-         where = frag - frag_now->fr_literal;
-         bfd_putl16 ((bfd_vma) bin, frag);
+         int x = op->exp.X_add_number;
 
-         if (op1.mode == OP_EXP)
+         if (vshift == 0)
            {
-             where += 2;
-             bfd_putl16 ((bfd_vma) ZEROS, frag + 2);
-
-             if (op1.reg)
-               fix_new_exp (frag_now, where, 2,
-                            &(op1.exp), FALSE, CHECK_RELOC_MSP430);
+             x = x & 0xffff;
+             op->exp.X_add_number = x;
+           }
+         else if (vshift == 1)
+           {
+             x = (x >> 16) & 0xffff;
+             op->exp.X_add_number = x;
+           }
+         else if (vshift > 1)
+           {
+             if (x < 0)
+               op->exp.X_add_number = -1;
              else
-               fix_new_exp (frag_now, where, 2,
-                            &(op1.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
+               op->exp.X_add_number = 0;       /* Nothing left.  */
+             x = op->exp.X_add_number;
            }
-         break;
-
-       case 2:
-         {
-           /* Shift instruction.  */
-           line = extract_operand (line, l1, sizeof (l1));
-           strncpy (l2, l1, sizeof (l2));
-           l2[sizeof (l2) - 1] = '\0';
-           res = msp430_srcoperand (&op1, l1, opcode->bin_opcode, &imm_op);
-           res += msp430_dstoperand (&op2, l2, opcode->bin_opcode);
-
-           if (res)
-             break;    /* An error occurred.  All warnings were done before.  */
-
-           bin |= (op2.reg | (op1.reg << 8) | (op1.am << 4) | (op2.am << 7));
-
-           __is = 1 + op1.ol + op2.ol; /* insn size in words.  */
-           frag = frag_more (2 * __is);
-           where = frag - frag_now->fr_literal;
-           bfd_putl16 ((bfd_vma) bin, frag);
-
-           if (op1.mode == OP_EXP)
-             {
-               where += 2;     /* Advance 'where' as we do not know _where_.  */
-               bfd_putl16 ((bfd_vma) ZEROS, frag + 2);
-
-               if (op1.reg || (op1.reg == 0 && op1.am == 3))   /* Not PC relative.  */
-                 fix_new_exp (frag_now, where, 2,
-                              &(op1.exp), FALSE, CHECK_RELOC_MSP430);
-               else
-                 fix_new_exp (frag_now, where, 2,
-                              &(op1.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
-             }
-
-           if (op2.mode == OP_EXP)
-             {
-               imm_op = 0;
-               bfd_putl16 ((bfd_vma) ZEROS, frag + 2 + ((__is == 3) ? 2 : 0));
-
-               if (op2.reg)    /* Not PC relative.  */
-                 fix_new_exp (frag_now, where + 2, 2,
-                              &(op2.exp), FALSE, CHECK_RELOC_MSP430);
-               else
-                 fix_new_exp (frag_now, where + 2, 2,
-                              &(op2.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
-             }
-           break;
-         }
-       case 3:
-         /* Branch instruction => mov dst, r0.  */
-         line = extract_operand (line, l1, sizeof (l1));
 
-         res = msp430_srcoperand (&op1, l1, opcode->bin_opcode, &imm_op);
-         if (res)
-           break;
+         if (op->exp.X_add_number > 65535 || op->exp.X_add_number < -32768)
+           {
+             as_bad (_("value %d out of range. Use #lo() or #hi()"), x);
+             return 1;
+           }
 
-         byte_op = 0;
-         imm_op = 0;
+         /* Now check constants.  */
+         /* Substitute register mode with a constant generator if applicable.  */
 
-         bin |= ((op1.reg << 8) | (op1.am << 4));
-         __is = 1 + op1.ol;
-         frag = frag_more (2 * __is);
-         where = frag - frag_now->fr_literal;
-         bfd_putl16 ((bfd_vma) bin, frag);
+         x = (short) x;        /* Extend sign.  */
 
-         if (op1.mode == OP_EXP)
+         if (x == 0)
            {
-             where += 2;
-             bfd_putl16 ((bfd_vma) ZEROS, frag + 2);
-
-             if (op1.reg || (op1.reg == 0 && op1.am == 3))
-               fix_new_exp (frag_now, where, 2,
-                            &(op1.exp), FALSE, CHECK_RELOC_MSP430);
-             else
-               fix_new_exp (frag_now, where, 2,
-                            &(op1.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
+             op->reg = 3;
+             op->am = 0;
+             op->ol = 0;
+             op->mode = OP_REG;
+           }
+         else if (x == 1)
+           {
+             op->reg = 3;
+             op->am = 1;
+             op->ol = 0;
+             op->mode = OP_REG;
+           }
+         else if (x == 2)
+           {
+             op->reg = 3;
+             op->am = 2;
+             op->ol = 0;
+             op->mode = OP_REG;
+           }
+         else if (x == -1)
+           {
+             op->reg = 3;
+             op->am = 3;
+             op->ol = 0;
+             op->mode = OP_REG;
+           }
+         else if (x == 4)
+           {
+#ifdef PUSH_1X_WORKAROUND
+             if (bin == 0x1200)
+               {
+                 /* Remove warning as confusing.
+                    as_warn(_("Hardware push bug workaround")); */
+               }
+             else
+#endif
+               {
+                 op->reg = 2;
+                 op->am = 2;
+                 op->ol = 0;
+                 op->mode = OP_REG;
+               }
+           }
+         else if (x == 8)
+           {
+#ifdef PUSH_1X_WORKAROUND
+             if (bin == 0x1200)
+               {
+                 /* Remove warning as confusing.
+                    as_warn(_("Hardware push bug workaround")); */
+               }
+             else
+#endif
+               {
+                 op->reg = 2;
+                 op->am = 3;
+                 op->ol = 0;
+                 op->mode = OP_REG;
+               }
            }
-         break;
        }
-      break;
-
-    case 1:                    /* Format 1, double operand.  */
-      line = extract_operand (line, l1, sizeof (l1));
-      line = extract_operand (line, l2, sizeof (l2));
-      res = msp430_srcoperand (&op1, l1, opcode->bin_opcode, &imm_op);
-      res += msp430_dstoperand (&op2, l2, opcode->bin_opcode);
+      else if (op->exp.X_op == O_symbol)
+       {
+         op->mode = OP_EXP;
+       }
+      else if (op->exp.X_op == O_big)
+       {
+         short x;
+         if (vshift != -1)
+           {
+             op->exp.X_op = O_constant;
+             op->exp.X_add_number = 0xffff & generic_bignum[vshift];
+             x = op->exp.X_add_number;
+           }
+         else
+           {
+             as_bad (_
+                     ("unknown expression in operand %s. use #llo() #lhi() #hlo() #hhi() "),
+                     l);
+             return 1;
+           }
 
-      if (res)
-       break;                  /* Error occurred.  All warnings were done before.  */
+         if (x == 0)
+           {
+             op->reg = 3;
+             op->am = 0;
+             op->ol = 0;
+             op->mode = OP_REG;
+           }
+         else if (x == 1)
+           {
+             op->reg = 3;
+             op->am = 1;
+             op->ol = 0;
+             op->mode = OP_REG;
+           }
+         else if (x == 2)
+           {
+             op->reg = 3;
+             op->am = 2;
+             op->ol = 0;
+             op->mode = OP_REG;
+           }
+         else if (x == -1)
+           {
+             op->reg = 3;
+             op->am = 3;
+             op->ol = 0;
+             op->mode = OP_REG;
+           }
+         else if (x == 4)
+           {
+             op->reg = 2;
+             op->am = 2;
+             op->ol = 0;
+             op->mode = OP_REG;
+           }
+         else if (x == 8)
+           {
+             op->reg = 2;
+             op->am = 3;
+             op->ol = 0;
+             op->mode = OP_REG;
+           }
+       }
+      /* Redudant (yet) check.  */
+      else if (op->exp.X_op == O_register)
+       as_bad
+         (_("Registers cannot be used within immediate expression [%s]"), l);
+      else
+       as_bad (_("unknown operand %s"), l);
 
-      bin |= (op2.reg | (op1.reg << 8) | (op1.am << 4) | (op2.am << 7));
+      return 0;
+    }
 
-      __is = 1 + op1.ol + op2.ol;      /* insn size in words.  */
-      frag = frag_more (2 * __is);
-      where = frag - frag_now->fr_literal;
-      bfd_putl16 ((bfd_vma) bin, frag);
+  /* Check if absolute &VALUE (assume that we can construct something like ((a&b)<<7 + 25).  */
+  if (*l == '&')
+    {
+      char *h = l;
 
-      if (op1.mode == OP_EXP)
+      op->reg = 2;             /* reg 2 in absolute addr mode.  */
+      op->am = 1;              /* mode As == 01 bin.  */
+      op->ol = 1;              /* Immediate value followed by instruction.  */
+      __tl = h + 1;
+      parse_exp (__tl, &(op->exp));
+      op->mode = OP_EXP;
+      if (op->exp.X_op == O_constant)
        {
-         where += 2;           /* Advance where as we do not know _where_.  */
-         bfd_putl16 ((bfd_vma) ZEROS, frag + 2);
+         int x = op->exp.X_add_number;
 
-         if (op1.reg || (op1.reg == 0 && op1.am == 3)) /* Not PC relative.  */
-           fix_new_exp (frag_now, where, 2,
-                        &(op1.exp), FALSE, CHECK_RELOC_MSP430);
-         else
-           fix_new_exp (frag_now, where, 2,
-                        &(op1.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
+         if (x > 65535 || x < -32768)
+           {
+             as_bad (_("value out of range: %d"), x);
+             return 1;
+           }
        }
-
-      if (op2.mode == OP_EXP)
+      else if (op->exp.X_op == O_symbol)
+       ;
+      else
        {
-         imm_op = 0;
-         bfd_putl16 ((bfd_vma) ZEROS, frag + 2 + ((__is == 3) ? 2 : 0));
-
-         if (op2.reg)          /* Not PC relative.  */
-           fix_new_exp (frag_now, where + 2, 2,
-                        &(op2.exp), FALSE, CHECK_RELOC_MSP430);
+         /* Redudant (yet) check.  */
+         if (op->exp.X_op == O_register)
+           as_bad
+             (_("Registers cannot be used within absolute expression [%s]"), l);
          else
-           fix_new_exp (frag_now, where + 2, 2,
-                        &(op2.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
+           as_bad (_("unknown expression in operand %s"), l);
+         return 1;
        }
-      break;
+      return 0;
+    }
 
-    case 2:                    /* Single-operand mostly instr.  */
-      if (opcode->insn_opnumb == 0)
+  /* Check if indirect register mode @Rn / postincrement @Rn+.  */
+  if (*l == '@')
+    {
+      char *t = l;
+      char *m = strchr (l, '+');
+
+      if (t != l)
        {
-         /* reti instruction.  */
-         frag = frag_more (2);
-         bfd_putl16 ((bfd_vma) bin, frag);
-         break;
+         as_bad (_("unknown addressing mode %s"), l);
+         return 1;
        }
 
-      line = extract_operand (line, l1, sizeof (l1));
-      res = msp430_srcoperand (&op1, l1, opcode->bin_opcode, &imm_op);
-      if (res)
-       break;          /* Error in operand.  */
+      t++;
+      if (*t != 'r' && *t != 'R')
+       {
+         as_bad (_("unknown addressing mode %s"), l);
+         return 1;
+       }
 
-      bin |= op1.reg | (op1.am << 4);
-      __is = 1 + op1.ol;
-      frag = frag_more (2 * __is);
-      where = frag - frag_now->fr_literal;
-      bfd_putl16 ((bfd_vma) bin, frag);
+      t++;     /* Points to the reg value.  */
 
-      if (op1.mode == OP_EXP)
+      if (check_reg (t))
        {
-         bfd_putl16 ((bfd_vma) ZEROS, frag + 2);
-
-         if (op1.reg || (op1.reg == 0 && op1.am == 3)) /* Not PC relative.  */
-           fix_new_exp (frag_now, where + 2, 2,
-                        &(op1.exp), FALSE, CHECK_RELOC_MSP430);
-         else
-           fix_new_exp (frag_now, where + 2, 2,
-                        &(op1.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
+         as_bad (_("Bad register name r%s"), t);
+         return 1;
        }
-      break;
 
-    case 3:                    /* Conditional jumps instructions.  */
-      line = extract_operand (line, l1, sizeof (l1));
-      /* l1 is a label.  */
-      if (l1[0])
+      op->mode = OP_REG;
+      op->am = m ? 3 : 2;
+      op->ol = 0;
+      if (m)
+       *m = 0;                 /* strip '+' */
+      op->reg = atoi (t);
+      if (op->reg < 0 || op->reg > 15)
        {
-         char *m = l1;
-         expressionS exp;
-
-         if (*m == '$')
-           m++;
+         as_bad (_("MSP430 does not have %d registers"), op->reg);
+         return 1;
+       }
 
-         parse_exp (m, &exp);
-         frag = frag_more (2); /* Instr size is 1 word.  */
+      return 0;
+    }
 
-         /* In order to handle something like:
+  /* Check if register indexed X(Rn).  */
+  do
+    {
+      char *h = strrchr (l, '(');
+      char *m = strrchr (l, ')');
+      char *t;
 
-            and #0x8000, r5
-            tst r5
-            jz   4     ;       skip next 4 bytes
-            inv r5
-            inc r5
-            nop        ;       will jump here if r5 positive or zero
+      *imm_op = 1;
 
-            jCOND      -n      ;assumes jump n bytes backward:
+      if (!h)
+       break;
+      if (!m)
+       {
+         as_bad (_("')' required"));
+         return 1;
+       }
 
-            mov r5,r6
-            jmp -2
+      t = h;
+      op->am = 1;
+      op->ol = 1;
+      /* Extract a register.  */
+      t++;     /* Advance pointer.  */
 
-            is equal to:
-            lab:
-            mov r5,r6
-            jmp lab
+      if (*t != 'r' && *t != 'R')
+       {
+         as_bad (_
+                 ("unknown operator %s. Did you mean X(Rn) or #[hl][hl][oi](CONST) ?"),
+                 l);
+         return 1;
+       }
+      t++;
 
-            jCOND      $n      ; jump from PC in either direction.  */
+      op->reg = *t - '0';
+      if (op->reg > 9 || op->reg < 0)
+       {
+         as_bad (_("unknown operator (r%s substituded as a register name"),
+                 t);
+         return 1;
+       }
+      t++;
+      if (*t != ')')
+       {
+         op->reg = op->reg * 10;
+         op->reg += *t - '0';
 
-         if (exp.X_op == O_constant)
+         if (op->reg > 15)
            {
-             int x = exp.X_add_number;
-
-             if (x & 1)
-               {
-                 as_warn (_("Even number required. Rounded to %d"), x + 1);
-                 x++;
-               }
-
-             if ((*l1 == '$' && x > 0) || x < 0)
-               x -= 2;
-
-             x >>= 1;
-
-             if (x > 512 || x < -511)
-               {
-                 as_bad (_("Wrong displacement  %d"), x << 1);
-                 break;
-               }
-
-             bin |= x & 0x3ff;
-             bfd_putl16 ((bfd_vma) bin, frag);
+             as_bad (_("unknown operator %s"), l);
+             return 1;
            }
-         else if (exp.X_op == O_symbol && *l1 != '$')
+         if (op->reg == 2)
            {
-             where = frag - frag_now->fr_literal;
-             fix_new_exp (frag_now, where, 2,
-                          &exp, TRUE, BFD_RELOC_MSP430_10_PCREL);
+             as_bad (_("r2 should not be used in indexed addressing mode"));
+             return 1;
+           }
 
-             bfd_putl16 ((bfd_vma) bin, frag);
+         if (*(t + 1) != ')')
+           {
+             as_bad (_("unknown operator %s"), l);
+             return 1;
            }
-         else if (*l1 == '$')
+       }
+
+      /* Extract constant.  */
+      __tl = l;
+      *h = 0;
+      op->mode = OP_EXP;
+      parse_exp (__tl, &(op->exp));
+      if (op->exp.X_op == O_constant)
+       {
+         int x = op->exp.X_add_number;
+
+         if (x > 65535 || x < -32768)
            {
-             as_bad (_("instruction requires label sans '$'"));
-             break;
+             as_bad (_("value out of range: %d"), x);
+             return 1;
            }
-         else
+
+         if (x == 0)
            {
-             as_bad (_
-                     ("instruction requires label or value in range -511:512"));
-             break;
+             op->mode = OP_REG;
+             op->am = 2;
+             op->ol = 0;
+             return 0;
            }
        }
+      else if (op->exp.X_op == O_symbol)
+       ;
       else
        {
-         as_bad (_("instruction requires label"));
-         break;
+         /* Redudant (yet) check.  */
+         if (op->exp.X_op == O_register)
+           as_bad
+             (_("Registers cannot be used as a prefix of indexed expression [%s]"), l);
+         else
+           as_bad (_("unknown expression in operand %s"), l);
+         return 1;
        }
-      break;
 
-    default:
-      as_bad (_("Ilegal instruction or not implmented opcode."));
+      return 0;
     }
+  while (0);
 
-  input_line_pointer = line;
-  return 0;
+  /* Register mode 'mov r1,r2'.  */
+  do
+    {
+      char *t = l;
+
+      /* Operand should be a register.  */
+      if (*t == 'r' || *t == 'R')
+       {
+         int x = atoi (t + 1);
+
+         if (check_reg (t + 1))
+           break;
+
+         if (x < 0 || x > 15)
+           break;              /* Symbolic mode.  */
+
+         op->mode = OP_REG;
+         op->am = 0;
+         op->ol = 0;
+         op->reg = x;
+         return 0;
+       }
+    }
+  while (0);
+
+  /* Symbolic mode 'mov a, b' == 'mov x(pc), y(pc)'.  */
+  do
+    {
+#if 0  /* Allow expression in operand like 'a+123*(1|2)'.  */
+      char *t = l;
+
+      __tl = l;
+
+      while (*t)
+       {
+         /* alpha/number    underline     dot for labels.  */
+         if (! ISALNUM (*t) && *t != '_' && *t != '.')
+           {
+             as_bad (_("unknown operand %s"), l);
+             return 1;
+           }
+         t++;
+       }
+#endif
+      op->mode = OP_EXP;
+      op->reg = 0;             /* PC relative... be careful.  */
+      op->am = 1;
+      op->ol = 1;
+      __tl = l;
+      parse_exp (__tl, &(op->exp));
+      return 0;
+    }
+  while (0);
+
+  /* Unreachable.  */
+  as_bad (_("unknown addressing mode for operand %s"), l);
+  return 1;
 }
 
+
 static int
-msp430_dstoperand (op, l, bin)
-     struct msp430_operand_s *op;
-     char *l;
-     int bin;
+msp430_dstoperand (struct msp430_operand_s * op, char * l, int bin)
 {
   int dummy;
-  int ret = msp430_srcoperand (op, l, bin, &dummy);
+  int ret = msp430_srcoperand (op, l, bin, & dummy);
+
   if (ret)
     return ret;
 
@@ -837,6 +1253,7 @@ msp430_dstoperand (op, l, bin)
       op->am = 1;
       op->ol = 1;
       parse_exp (__tl, &(op->exp));
+
       if (op->exp.X_op != O_constant || op->exp.X_add_number != 0)
        {
          as_bad (_("Internal bug. Try to use 0(r%d) instead of @r%d"),
@@ -856,497 +1273,449 @@ msp430_dstoperand (op, l, bin)
 }
 
 
-static int
-check_reg (t)
-     char *t;
+/* Parse instruction operands.
+   Return binary opcode.  */
+
+static unsigned int
+msp430_operands (struct msp430_opcode_s * opcode, char * line)
 {
-  /* If this is a reg numb, str 't' must be a number from 0 - 15.  */
+  int bin = opcode->bin_opcode;        /* Opcode mask.  */
+  int __is;
+  char l1[MAX_OP_LEN], l2[MAX_OP_LEN];
+  char *frag;
+  int where;
+  struct msp430_operand_s op1, op2;
+  int res = 0;
+  static short ZEROS = 0;
+  int byte_op, imm_op;
 
-  if (strlen (t) > 2 && *(t + 2) != '+')
-    return 1;
+  /* Opcode is the one from opcodes table
+     line contains something like
+     [.w] @r2+, 5(R1)
+     or
+     .b @r2+, 5(R1).  */
 
-  while (*t)
+  /* Check if byte or word operation.  */
+  if (*line == '.' && TOLOWER (*(line + 1)) == 'b')
     {
-      if ((*t < '0' || *t > '9') && *t != '+')
-       break;
-      t++;
+      bin |= BYTE_OPERATION;
+      byte_op = 1;
     }
+  else
+    byte_op = 0;
 
-  if (*t)
-    return 1;
-
-  return 0;
-}
-
-
-static int
-msp430_srcoperand (op, l, bin, imm_op)
-     struct msp430_operand_s *op;
-     char *l;
-     int bin;
-     int *imm_op;
-{
-  char *__tl = l;
+  /* skip .[bwBW].  */
+  while (! ISSPACE (*line) && *line)
+    line++;
 
-  /* Check if an immediate #VALUE.  The hash sign should be only at the beginning!  */
-  if (*l == '#')
+  if (opcode->insn_opnumb && (!*line || *line == '\n'))
     {
-      char *h = l;
-      int vshift = -1;
-      int rval = 0;
+      as_bad (_("instruction %s requires %d operand(s)"),
+             opcode->name, opcode->insn_opnumb);
+      return 0;
+    }
 
-      /* Check if there is:
-        llo(x) - least significant 16 bits, x &= 0xffff
-        lhi(x) - x = (x >> 16) & 0xffff,
-        hlo(x) - x = (x >> 32) & 0xffff,
-        hhi(x) - x = (x >> 48) & 0xffff
-        The value _MUST_ be constant expression: #hlo(1231231231).  */
+  memset (l1, 0, sizeof (l1));
+  memset (l2, 0, sizeof (l2));
+  memset (&op1, 0, sizeof (op1));
+  memset (&op2, 0, sizeof (op2));
 
-      *imm_op = 1;
+  imm_op = 0;
 
-      if (strncasecmp (h, "#llo(", 5) == 0)
-       {
-         vshift = 0;
-         rval = 3;
-       }
-      else if (strncasecmp (h, "#lhi(", 5) == 0)
-       {
-         vshift = 1;
-         rval = 3;
-       }
-      else if (strncasecmp (h, "#hlo(", 5) == 0)
-       {
-         vshift = 2;
-         rval = 3;
-       }
-      else if (strncasecmp (h, "#hhi(", 5) == 0)
-       {
-         vshift = 3;
-         rval = 3;
-       }
-      else if (strncasecmp (h, "#lo(", 4) == 0)
-       {
-         vshift = 0;
-         rval = 2;
-       }
-      else if (strncasecmp (h, "#hi(", 4) == 0)
+  switch (opcode->fmt)
+    {
+    case 0:                    /* Emulated.  */
+      switch (opcode->insn_opnumb)
        {
-         vshift = 1;
-         rval = 2;
-       }
+       case 0:
+         /* Set/clear bits instructions.  */
+         __is = 2;
+         frag = frag_more (__is);
+         bfd_putl16 ((bfd_vma) bin, frag);
+         break;
+       case 1:
+         /* Something which works with destination operand.  */
+         line = extract_operand (line, l1, sizeof (l1));
+         res = msp430_dstoperand (&op1, l1, opcode->bin_opcode);
+         if (res)
+           break;
 
-      op->reg = 0;             /* Reg PC.  */
-      op->am = 3;
-      op->ol = 1;              /* Immediate  will follow an instruction.  */
-      __tl = h + 1 + rval;
-      op->mode = OP_EXP;
-      parse_exp (__tl, &(op->exp));
-      if (op->exp.X_op == O_constant)
-       {
-         int x = op->exp.X_add_number;
+         bin |= (op1.reg | (op1.am << 7));
+         __is = 1 + op1.ol;
+         frag = frag_more (2 * __is);
+         where = frag - frag_now->fr_literal;
+         bfd_putl16 ((bfd_vma) bin, frag);
 
-         if (vshift == 0)
-           {
-             x = x & 0xffff;
-             op->exp.X_add_number = x;
-           }
-         else if (vshift == 1)
-           {
-             x = (x >> 16) & 0xffff;
-             op->exp.X_add_number = x;
-           }
-         else if (vshift > 1)
+         if (op1.mode == OP_EXP)
            {
-             if (x < 0)
-               op->exp.X_add_number = -1;
+             where += 2;
+             bfd_putl16 ((bfd_vma) ZEROS, frag + 2);
+
+             if (op1.reg)
+               fix_new_exp (frag_now, where, 2,
+                            &(op1.exp), FALSE, CHECK_RELOC_MSP430);
              else
-               op->exp.X_add_number = 0;       /* Nothing left.  */
-             x = op->exp.X_add_number;
+               fix_new_exp (frag_now, where, 2,
+                            &(op1.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
            }
+         break;
 
-         if (op->exp.X_add_number > 65535 || op->exp.X_add_number < -32768)
-           {
-             as_bad (_("value %ld out of range. Use #lo() or #hi()"), x);
-             return 1;
-           }
+       case 2:
+         {
+           /* Shift instruction.  */
+           line = extract_operand (line, l1, sizeof (l1));
+           strncpy (l2, l1, sizeof (l2));
+           l2[sizeof (l2) - 1] = '\0';
+           res = msp430_srcoperand (&op1, l1, opcode->bin_opcode, &imm_op);
+           res += msp430_dstoperand (&op2, l2, opcode->bin_opcode);
 
-         /* Now check constants.  */
-         /* Substitute register mode with a constant generator if applicable.  */
+           if (res)
+             break;    /* An error occurred.  All warnings were done before.  */
 
-         x = (short) x;        /* Extend sign.  */
+           bin |= (op2.reg | (op1.reg << 8) | (op1.am << 4) | (op2.am << 7));
 
-         if (x == 0)
-           {
-             op->reg = 3;
-             op->am = 0;
-             op->ol = 0;
-             op->mode = OP_REG;
-           }
-         else if (x == 1)
-           {
-             op->reg = 3;
-             op->am = 1;
-             op->ol = 0;
-             op->mode = OP_REG;
-           }
-         else if (x == 2)
-           {
-             op->reg = 3;
-             op->am = 2;
-             op->ol = 0;
-             op->mode = OP_REG;
-           }
-         else if (x == -1)
-           {
-             op->reg = 3;
-             op->am = 3;
-             op->ol = 0;
-             op->mode = OP_REG;
-           }
-         else if (x == 4)
-           {
-#ifdef PUSH_1X_WORKAROUND
-             if (bin == 0x1200)
-               {
-                 /* Remove warning as confusing.
-                    as_warn(_("Hardware push bug workaround")); */
-               }
-             else
-#endif
-               {
-                 op->reg = 2;
-                 op->am = 2;
-                 op->ol = 0;
-                 op->mode = OP_REG;
-               }
-           }
-         else if (x == 8)
+           __is = 1 + op1.ol + op2.ol; /* insn size in words.  */
+           frag = frag_more (2 * __is);
+           where = frag - frag_now->fr_literal;
+           bfd_putl16 ((bfd_vma) bin, frag);
+
+           if (op1.mode == OP_EXP)
+             {
+               where += 2;     /* Advance 'where' as we do not know _where_.  */
+               bfd_putl16 ((bfd_vma) ZEROS, frag + 2);
+
+               if (op1.reg || (op1.reg == 0 && op1.am == 3))   /* Not PC relative.  */
+                 fix_new_exp (frag_now, where, 2,
+                              &(op1.exp), FALSE, CHECK_RELOC_MSP430);
+               else
+                 fix_new_exp (frag_now, where, 2,
+                              &(op1.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
+             }
+
+           if (op2.mode == OP_EXP)
+             {
+               imm_op = 0;
+               bfd_putl16 ((bfd_vma) ZEROS, frag + 2 + ((__is == 3) ? 2 : 0));
+
+               if (op2.reg)    /* Not PC relative.  */
+                 fix_new_exp (frag_now, where + 2, 2,
+                              &(op2.exp), FALSE, CHECK_RELOC_MSP430);
+               else
+                 fix_new_exp (frag_now, where + 2, 2,
+                              &(op2.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
+             }
+           break;
+         }
+       case 3:
+         /* Branch instruction => mov dst, r0.  */
+         line = extract_operand (line, l1, sizeof (l1));
+
+         res = msp430_srcoperand (&op1, l1, opcode->bin_opcode, &imm_op);
+         if (res)
+           break;
+
+         byte_op = 0;
+         imm_op = 0;
+
+         bin |= ((op1.reg << 8) | (op1.am << 4));
+         __is = 1 + op1.ol;
+         frag = frag_more (2 * __is);
+         where = frag - frag_now->fr_literal;
+         bfd_putl16 ((bfd_vma) bin, frag);
+
+         if (op1.mode == OP_EXP)
            {
-#ifdef PUSH_1X_WORKAROUND
-             if (bin == 0x1200)
-               {
-                 /* Remove warning as confusing.
-                    as_warn(_("Hardware push bug workaround")); */
-               }
+             where += 2;
+             bfd_putl16 ((bfd_vma) ZEROS, frag + 2);
+
+             if (op1.reg || (op1.reg == 0 && op1.am == 3))
+               fix_new_exp (frag_now, where, 2,
+                            &(op1.exp), FALSE, CHECK_RELOC_MSP430);
              else
-#endif
-               {
-                 op->reg = 2;
-                 op->am = 3;
-                 op->ol = 0;
-                 op->mode = OP_REG;
-               }
+               fix_new_exp (frag_now, where, 2,
+                            &(op1.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
            }
+         break;
        }
-      else if (op->exp.X_op == O_symbol)
-       {
-         op->mode = OP_EXP;
-       }
-      else if (op->exp.X_op == O_big)
-       {
-         short x;
-         if (vshift != -1)
-           {
-             op->exp.X_op = O_constant;
-             op->exp.X_add_number = 0xffff & generic_bignum[vshift];
-             x = op->exp.X_add_number;
-           }
-         else
-           {
-             as_bad (_
-                     ("unknown expression in operand %s. use #llo() #lhi() #hlo() #hhi() "),
-                     l);
-             return 1;
-           }
+      break;
 
-         if (x == 0)
-           {
-             op->reg = 3;
-             op->am = 0;
-             op->ol = 0;
-             op->mode = OP_REG;
-           }
-         else if (x == 1)
-           {
-             op->reg = 3;
-             op->am = 1;
-             op->ol = 0;
-             op->mode = OP_REG;
-           }
-         else if (x == 2)
-           {
-             op->reg = 3;
-             op->am = 2;
-             op->ol = 0;
-             op->mode = OP_REG;
-           }
-         else if (x == -1)
-           {
-             op->reg = 3;
-             op->am = 3;
-             op->ol = 0;
-             op->mode = OP_REG;
-           }
-         else if (x == 4)
-           {
-             op->reg = 2;
-             op->am = 2;
-             op->ol = 0;
-             op->mode = OP_REG;
-           }
-         else if (x == 8)
-           {
-             op->reg = 2;
-             op->am = 3;
-             op->ol = 0;
-             op->mode = OP_REG;
-           }
-       }
-      else
-       {
-         as_bad (_("unknown operand %s"), l);
-       }
-      return 0;
-    }
+    case 1:                    /* Format 1, double operand.  */
+      line = extract_operand (line, l1, sizeof (l1));
+      line = extract_operand (line, l2, sizeof (l2));
+      res = msp430_srcoperand (&op1, l1, opcode->bin_opcode, &imm_op);
+      res += msp430_dstoperand (&op2, l2, opcode->bin_opcode);
 
-  /* Check if absolute &VALUE (assume that we can construct something like ((a&b)<<7 + 25).  */
-  if (*l == '&')
-    {
-      char *h = l;
+      if (res)
+       break;                  /* Error occurred.  All warnings were done before.  */
 
-      op->reg = 2;             /* reg 2 in absolute addr mode.  */
-      op->am = 1;              /* mode As == 01 bin.  */
-      op->ol = 1;              /* Immediate value followed by instruction.  */
-      __tl = h + 1;
-      parse_exp (__tl, &(op->exp));
-      op->mode = OP_EXP;
-      if (op->exp.X_op == O_constant)
+      bin |= (op2.reg | (op1.reg << 8) | (op1.am << 4) | (op2.am << 7));
+
+      __is = 1 + op1.ol + op2.ol;      /* insn size in words.  */
+      frag = frag_more (2 * __is);
+      where = frag - frag_now->fr_literal;
+      bfd_putl16 ((bfd_vma) bin, frag);
+
+      if (op1.mode == OP_EXP)
        {
-         int x = op->exp.X_add_number;
-         if (x > 65535 || x < -32768)
-           {
-             as_bad (_("value out of range: %d"), x);
-             return 1;
-           }
+         where += 2;           /* Advance where as we do not know _where_.  */
+         bfd_putl16 ((bfd_vma) ZEROS, frag + 2);
+
+         if (op1.reg || (op1.reg == 0 && op1.am == 3)) /* Not PC relative.  */
+           fix_new_exp (frag_now, where, 2,
+                        &(op1.exp), FALSE, CHECK_RELOC_MSP430);
+         else
+           fix_new_exp (frag_now, where, 2,
+                        &(op1.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
        }
-      else if (op->exp.X_op == O_symbol)
+
+      if (op2.mode == OP_EXP)
        {
+         imm_op = 0;
+         bfd_putl16 ((bfd_vma) ZEROS, frag + 2 + ((__is == 3) ? 2 : 0));
+
+         if (op2.reg)          /* Not PC relative.  */
+           fix_new_exp (frag_now, where + 2, 2,
+                        &(op2.exp), FALSE, CHECK_RELOC_MSP430);
+         else
+           fix_new_exp (frag_now, where + 2, 2,
+                        &(op2.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
        }
-      else
+      break;
+
+    case 2:                    /* Single-operand mostly instr.  */
+      if (opcode->insn_opnumb == 0)
        {
-         as_bad (_("unknown expression in operand %s"), l);
-         return 1;
+         /* reti instruction.  */
+         frag = frag_more (2);
+         bfd_putl16 ((bfd_vma) bin, frag);
+         break;
        }
-      return 0;
-    }
 
-  /* Check if indirect register mode @Rn / postincrement @Rn+.  */
-  if (*l == '@')
-    {
-      char *t = l;
-      char *m = strchr (l, '+');
+      line = extract_operand (line, l1, sizeof (l1));
+      res = msp430_srcoperand (&op1, l1, opcode->bin_opcode, &imm_op);
+      if (res)
+       break;          /* Error in operand.  */
 
-      if (t != l)
+      bin |= op1.reg | (op1.am << 4);
+      __is = 1 + op1.ol;
+      frag = frag_more (2 * __is);
+      where = frag - frag_now->fr_literal;
+      bfd_putl16 ((bfd_vma) bin, frag);
+
+      if (op1.mode == OP_EXP)
        {
-         as_bad (_("unknown addressing mode %s"), l);
-         return 1;
+         bfd_putl16 ((bfd_vma) ZEROS, frag + 2);
+
+         if (op1.reg || (op1.reg == 0 && op1.am == 3)) /* Not PC relative.  */
+           fix_new_exp (frag_now, where + 2, 2,
+                        &(op1.exp), FALSE, CHECK_RELOC_MSP430);
+         else
+           fix_new_exp (frag_now, where + 2, 2,
+                        &(op1.exp), TRUE, CHECK_RELOC_MSP430_PCREL);
        }
+      break;
 
-      t++;
-      if (*t != 'r' && *t != 'R')
+    case 3:                    /* Conditional jumps instructions.  */
+      line = extract_operand (line, l1, sizeof (l1));
+      /* l1 is a label.  */
+      if (l1[0])
        {
-         as_bad (_("unknown addressing mode %s"), l);
-         return 1;
-       }
+         char *m = l1;
+         expressionS exp;
 
-      t++;     /* Points to the reg value.  */
+         if (*m == '$')
+           m++;
 
-      if (check_reg (t))
-       {
-         as_bad (_("Bad register name r%s"), t);
-         return 1;
-       }
+         parse_exp (m, &exp);
+         frag = frag_more (2); /* Instr size is 1 word.  */
 
-      op->mode = OP_REG;
-      op->am = m ? 3 : 2;
-      op->ol = 0;
-      if (m)
-       *m = 0;                 /* strip '+' */
-      op->reg = atoi (t);
-      if (op->reg < 0 || op->reg > 15)
-       {
-         as_bad (_("MSP430 does not have %d registers"), op->reg);
-         return 1;
-       }
+         /* In order to handle something like:
 
-      return 0;
-    }
+            and #0x8000, r5
+            tst r5
+            jz   4     ;       skip next 4 bytes
+            inv r5
+            inc r5
+            nop        ;       will jump here if r5 positive or zero
 
-  /* Check if register indexed X(Rn).  */
-  do
-    {
-      char *h = strrchr (l, '(');
-      char *m = strrchr (l, ')');
-      char *t;
+            jCOND      -n      ;assumes jump n bytes backward:
 
-      *imm_op = 1;
+            mov r5,r6
+            jmp -2
 
-      if (!h)
-       break;
-      if (!m)
-       {
-         as_bad (_("')' required"));
-         return 1;
-       }
+            is equal to:
+            lab:
+            mov r5,r6
+            jmp lab
+
+            jCOND      $n      ; jump from PC in either direction.  */
 
-      t = h;
-      op->am = 1;
-      op->ol = 1;
-      /* Extract a register.  */
-      t++;     /* Advance pointer.  */
+         if (exp.X_op == O_constant)
+           {
+             int x = exp.X_add_number;
 
-      if (*t != 'r' && *t != 'R')
-       {
-         as_bad (_
-                 ("unknown operator %s. Did you mean X(Rn) or #[hl][hl][oi](CONST) ?"),
-                 l);
-         return 1;
-       }
-      t++;
+             if (x & 1)
+               {
+                 as_warn (_("Even number required. Rounded to %d"), x + 1);
+                 x++;
+               }
 
-      op->reg = *t - '0';
-      if (op->reg > 9 || op->reg < 0)
-       {
-         as_bad (_("unknown operator (r%s substituded as a register name"),
-                 t);
-         return 1;
-       }
-      t++;
-      if (*t != ')')
-       {
-         op->reg = op->reg * 10;
-         op->reg += *t - '0';
+             if ((*l1 == '$' && x > 0) || x < 0)
+               x -= 2;
 
-         if (op->reg > 15)
+             x >>= 1;
+
+             if (x > 512 || x < -511)
+               {
+                 as_bad (_("Wrong displacement  %d"), x << 1);
+                 break;
+               }
+
+             bin |= x & 0x3ff;
+             bfd_putl16 ((bfd_vma) bin, frag);
+           }
+         else if (exp.X_op == O_symbol && *l1 != '$')
            {
-             as_bad (_("unknown operator %s"), l);
-             return 1;
+             where = frag - frag_now->fr_literal;
+             fix_new_exp (frag_now, where, 2,
+                          &exp, TRUE, BFD_RELOC_MSP430_10_PCREL);
+
+             bfd_putl16 ((bfd_vma) bin, frag);
            }
-         if (op->reg == 2)
+         else if (*l1 == '$')
            {
-             as_bad (_("r2 should not be used in indexed addressing mode"));
-             return 1;
+             as_bad (_("instruction requires label sans '$'"));
+             break;
            }
-
-         if (*(t + 1) != ')')
+         else
            {
-             as_bad (_("unknown operator %s"), l);
-             return 1;
+             as_bad (_
+                     ("instruction requires label or value in range -511:512"));
+             break;
            }
        }
+      else
+       {
+         as_bad (_("instruction requires label"));
+         break;
+       }
+      break;
 
-      /* Extract constant.  */
-      __tl = l;
-      *h = 0;
-      op->mode = OP_EXP;
-      parse_exp (__tl, &(op->exp));
-      if (op->exp.X_op == O_constant)
+    case 4:    /* Extended jumps.  */
+      line = extract_operand (line, l1, sizeof (l1));
+      if (l1[0])
        {
-         int x = op->exp.X_add_number;
+         char *m = l1;
+         expressionS exp;
 
-         if (x > 65535 || x < -32768)
-           {
-             as_bad (_("value out of range: %d"), x);
-             return 1;
-           }
+         /* Ignore absolute addressing. make it PC relative anyway.  */
+         if (*m == '#' || *m == '$')
+           m++;
 
-         if (x == 0)
+         parse_exp (m, & exp);
+         if (exp.X_op == O_symbol)
            {
-             op->mode = OP_REG;
-             op->am = 2;
-             op->ol = 0;
-             return 0;
+             /* Relaxation required.  */
+             struct rcodes_s rc = msp430_rcodes[opcode->insn_opnumb];
+
+             frag = frag_more (8);
+             bfd_putl16 ((bfd_vma) rc.sop, frag);
+             frag = frag_variant (rs_machine_dependent, 8, 2,
+                                  ENCODE_RELAX (rc.lpos, STATE_BITS10), /* Wild guess.  */
+                                  exp.X_add_symbol,
+                                  0,   /* Offset is zero if jump dist less than 1K.  */
+                                  (char *) frag);
+             break;
            }
        }
-      else if (op->exp.X_op == O_symbol)
-       {
-       }
-      else
+
+      as_bad (_("instruction requires label"));
+      break;
+
+    case 5:    /* Emulated extended branches.  */
+      line = extract_operand (line, l1, sizeof (l1));
+      if (l1[0])
        {
-         as_bad (_("unknown expression in operand %s"), l);
-         return 1;
+         char * m = l1;
+         expressionS exp;
+
+         /* Ignore absolute addressing. make it PC relative anyway.  */
+         if (*m == '#' || *m == '$')
+           m++;
+
+         parse_exp (m, & exp);
+         if (exp.X_op == O_symbol)
+           {
+             /* Relaxation required.  */
+             struct hcodes_s hc = msp430_hcodes[opcode->insn_opnumb];
+
+             frag = frag_more (8);
+             bfd_putl16 ((bfd_vma) hc.op0, frag);
+             bfd_putl16 ((bfd_vma) hc.op1, frag+2);
+             frag = frag_variant (rs_machine_dependent, 8, 2,
+                                  ENCODE_RELAX (STATE_EMUL_BRANCH, STATE_BITS10), /* Wild guess.  */
+                                  exp.X_add_symbol,
+                                  0,   /* Offset is zero if jump dist less than 1K.  */
+                                  (char *) frag);
+             break;
+           }
        }
 
-      return 0;
-    }
-  while (0);
+      as_bad (_("instruction requires label"));
+      break;
 
-  /* Register mode 'mov r1,r2'.  */
-  do
-    {
-      char *t = l;
+    default:
+      as_bad (_("Ilegal instruction or not implmented opcode."));
+    }
 
-      /* Operand should be a register.  */
-      if (*t == 'r' || *t == 'R')
-       {
-         int x = atoi (t + 1);
+  input_line_pointer = line;
+  return 0;
+}
 
-         if (check_reg (t + 1))
-           break;
+void
+md_assemble (char * str)
+{
+  struct msp430_opcode_s * opcode;
+  char cmd[32];
+  unsigned int i = 0;
 
-         if (x < 0 || x > 15)
-           break;              /* Symbolic mode.  */
+  str = skip_space (str);      /* Skip leading spaces.  */
+  str = extract_cmd (str, cmd, sizeof (cmd));
 
-         op->mode = OP_REG;
-         op->am = 0;
-         op->ol = 0;
-         op->reg = x;
-         return 0;
-       }
+  while (cmd[i] && i < sizeof (cmd))
+    {
+      char a = TOLOWER (cmd[i]);
+      cmd[i] = a;
+      i++;
     }
-  while (0);
 
-  /* Symbolic mode 'mov a, b' == 'mov x(pc), y(pc)'.  */
-  do
+  if (!cmd[0])
     {
-      char *t = l;
-
-      __tl = l;
+      as_bad (_("can't find opcode "));
+      return;
+    }
 
-      while (*t)
-       {
-         /* alpha/number    underline     dot for labels.  */
-         if (! ISALNUM (*t) && *t != '_' && *t != '.')
-           {
-             as_bad (_("unknown operand %s"), l);
-             return 1;
-           }
-         t++;
-       }
+  opcode = (struct msp430_opcode_s *) hash_find (msp430_hash, cmd);
 
-      op->mode = OP_EXP;
-      op->reg = 0;             /* PC relative... be careful.  */
-      op->am = 1;
-      op->ol = 1;
-      __tl = l;
-      parse_exp (__tl, &(op->exp));
-      return 0;
+  if (opcode == NULL)
+    {
+      as_bad (_("unknown opcode `%s'"), cmd);
+      return;
     }
-  while (0);
 
-  /* Unreachable.  */
-  as_bad (_("unknown addressing mode for operand %s"), l);
-  return 1;
-}
+  {
+    char *__t = input_line_pointer;
 
+    msp430_operands (opcode, str);
+    input_line_pointer = __t;
+  }
+}
 
 /* GAS will call this function for each section at the end of the assembly,
    to permit the CPU backend to adjust the alignment of a section.  */
 
 valueT
-md_section_align (seg, addr)
-     asection *seg;
-     valueT addr;
+md_section_align (asection * seg, valueT addr)
 {
   int align = bfd_get_section_alignment (stdoutput, seg);
 
@@ -1360,9 +1729,7 @@ md_section_align (seg, addr)
    macro would return the length of an instruction.  */
 
 long
-md_pcrel_from_section (fixp, sec)
-     fixS *fixp;
-     segT sec;
+md_pcrel_from_section (fixS * fixp, segT sec)
 {
   if (fixp->fx_addsy != (symbolS *) NULL
       && (!S_IS_DEFINED (fixp->fx_addsy)
@@ -1376,12 +1743,9 @@ md_pcrel_from_section (fixp, sec)
    value in the object file.  */
 
 void
-md_apply_fix3 (fixp, valuep, seg)
-     fixS *fixp;
-     valueT *valuep;
-     segT seg;
+md_apply_fix3 (fixS * fixp, valueT * valuep, segT seg)
 {
-  unsigned char *where;
+  unsigned char * where;
   unsigned long insn;
   long value;
 
@@ -1459,6 +1823,7 @@ md_apply_fix3 (fixp, valuep, seg)
          bfd_putl16 ((bfd_vma) (value | insn), where);
          break;
 
+       case BFD_RELOC_MSP430_RL_PCREL:
        case BFD_RELOC_MSP430_16_PCREL:
          if (value & 1)
            as_bad_where (fixp->fx_file, fixp->fx_line,
@@ -1517,15 +1882,13 @@ md_apply_fix3 (fixp, valuep, seg)
    then it is done here.  */
 
 arelent *
-tc_gen_reloc (seg, fixp)
-     asection *seg ATTRIBUTE_UNUSED;
-     fixS *fixp;
+tc_gen_reloc (asection * seg ATTRIBUTE_UNUSED, fixS * fixp)
 {
-  arelent *reloc;
+  arelent * reloc;
 
-  reloc = (arelent *) xmalloc (sizeof (arelent));
+  reloc = xmalloc (sizeof (arelent));
 
-  reloc->sym_ptr_ptr = (asymbol **) xmalloc (sizeof (asymbol *));
+  reloc->sym_ptr_ptr = xmalloc (sizeof (asymbol *));
   *reloc->sym_ptr_ptr = symbol_get_bfdsym (fixp->fx_addsy);
 
   reloc->address = fixp->fx_frag->fr_address + fixp->fx_where;
@@ -1547,26 +1910,223 @@ tc_gen_reloc (seg, fixp)
   return reloc;
 }
 
-/* Parse ordinary expression.  */
+int
+md_estimate_size_before_relax (fragS * fragP ATTRIBUTE_UNUSED,
+                              asection * segment_type ATTRIBUTE_UNUSED)
+{
+  if (fragP->fr_symbol && S_GET_SEGMENT (fragP->fr_symbol) == segment_type)
+    {
+      /* This is a jump -> pcrel mode. Nothing to do much here.
+         Return value == 2.  */
+      fragP->fr_subtype =
+         ENCODE_RELAX (RELAX_LEN (fragP->fr_subtype), STATE_BITS10);
+    }
+  else if (fragP->fr_symbol)
+    {
+      /* Its got a segment, but its not ours.   Even if fr_symbol is in
+        an absolute segment, we dont know a displacement until we link
+        object files. So it will always be long. This also applies to
+        labels in a subsegment of current. Liker may relax it to short
+        jump later. Return value == 8.  */
+      fragP->fr_subtype =
+         ENCODE_RELAX (RELAX_LEN (fragP->fr_subtype), STATE_WORD);
+    }
+  else
+    {
+      /* We know the abs value. may be it is a jump to fixed address.
+         Impossible in our case, cause all constants already handeled. */
+      fragP->fr_subtype =
+         ENCODE_RELAX (RELAX_LEN (fragP->fr_subtype), STATE_UNDEF);
+    }
 
-static char *
-parse_exp (s, op)
-     char *s;
-     expressionS *op;
+  return md_relax_table[fragP->fr_subtype].rlx_length;
+}
+
+void
+md_convert_frag (bfd * abfd ATTRIBUTE_UNUSED,
+                asection * sec ATTRIBUTE_UNUSED,
+                fragS * fragP)
 {
-  input_line_pointer = s;
-  expression (op);
-  if (op->X_op == O_absent)
-    as_bad (_("missing operand"));
-  return input_line_pointer;
+  char * where = 0;
+  int rela = -1;
+  int i;
+  struct rcodes_s * cc = NULL;
+  struct hcodes_s * hc = NULL;
+
+  switch (fragP->fr_subtype)
+    {
+    case ENCODE_RELAX (STATE_UNCOND_BRANCH, STATE_BITS10):
+    case ENCODE_RELAX (STATE_SIMPLE_BRANCH, STATE_BITS10):
+    case ENCODE_RELAX (STATE_NOOV_BRANCH, STATE_BITS10):
+      /* We do not have to convert anything here.
+         Just apply a fix.  */
+      rela = BFD_RELOC_MSP430_10_PCREL;
+      break;
+
+    case ENCODE_RELAX (STATE_UNCOND_BRANCH, STATE_WORD):
+    case ENCODE_RELAX (STATE_UNCOND_BRANCH, STATE_UNDEF):
+      /* Convert uncond branch jmp lab -> br lab.  */
+      cc = & msp430_rcodes[7];
+      where = fragP->fr_literal + fragP->fr_fix;
+      bfd_putl16 (cc->lop0, where);
+      rela = BFD_RELOC_MSP430_RL_PCREL;
+      fragP->fr_fix += 2;
+      break;
+
+    case ENCODE_RELAX (STATE_SIMPLE_BRANCH, STATE_WORD):
+    case ENCODE_RELAX (STATE_SIMPLE_BRANCH, STATE_UNDEF):
+      {
+       /* Other simple branches.  */
+       int insn = bfd_getl16 (fragP->fr_opcode);
+
+       insn &= 0xffff;
+       /* Find actual instruction.  */
+       for (i = 0; i < 7 && !cc; i++)
+         if (msp430_rcodes[i].sop == insn)
+           cc = & msp430_rcodes[i];
+       if (!cc || !cc->name)
+         as_fatal (_("internal inconsistency problem in %s: insn %04lx"),
+                   __FUNCTION__, (long) insn);
+       where = fragP->fr_literal + fragP->fr_fix;
+       bfd_putl16 (cc->lop0, where);
+       bfd_putl16 (cc->lop1, where + 2);
+       rela = BFD_RELOC_MSP430_RL_PCREL;
+       fragP->fr_fix += 4;
+      }
+      break;
+
+    case ENCODE_RELAX (STATE_NOOV_BRANCH, STATE_WORD):
+    case ENCODE_RELAX (STATE_NOOV_BRANCH, STATE_UNDEF):
+      cc = & msp430_rcodes[6];
+      where = fragP->fr_literal + fragP->fr_fix;
+      bfd_putl16 (cc->lop0, where);
+      bfd_putl16 (cc->lop1, where + 2);
+      bfd_putl16 (cc->lop2, where + 4);
+      rela = BFD_RELOC_MSP430_RL_PCREL;
+      fragP->fr_fix += 6;
+      break;
+
+    case ENCODE_RELAX (STATE_EMUL_BRANCH, STATE_BITS10):
+      {
+       int insn = bfd_getl16 (fragP->fr_opcode + 2);
+
+       insn &= 0xffff;
+       for (i = 0; i < 4 && !hc; i++)
+         if (msp430_hcodes[i].op1 == insn)
+           hc = &msp430_hcodes[i];
+       if (!hc || !hc->name)
+         as_fatal (_("internal inconsistency problem in %s: ext. insn %04lx"),
+             __FUNCTION__, (long) insn);
+       rela = BFD_RELOC_MSP430_10_PCREL;
+       /* Apply a fix for a first label if necessary.
+          another fix will be applied to the next word of insn anyway.  */
+       if (hc->tlab == 2)
+         fix_new (fragP, fragP->fr_fix, 2, fragP->fr_symbol,
+             fragP->fr_offset, TRUE, rela);
+       fragP->fr_fix += 2;
+      }
+
+      break;
+
+    case ENCODE_RELAX (STATE_EMUL_BRANCH, STATE_WORD):
+    case ENCODE_RELAX (STATE_EMUL_BRANCH, STATE_UNDEF):
+      {
+       int insn = bfd_getl16 (fragP->fr_opcode + 2);
+
+       insn &= 0xffff;
+       for (i = 0; i < 4 && !hc; i++)
+         if (msp430_hcodes[i].op1 == insn)
+           hc = & msp430_hcodes[i];
+       if (!hc || !hc->name)
+         as_fatal (_("internal inconsistency problem in %s: ext. insn %04lx"),
+             __FUNCTION__, (long) insn);
+       rela = BFD_RELOC_MSP430_RL_PCREL;
+       where = fragP->fr_literal + fragP->fr_fix;
+       bfd_putl16 (hc->lop0, where);
+       bfd_putl16 (hc->lop1, where + 2);
+       bfd_putl16 (hc->lop2, where + 4);
+       fragP->fr_fix += 6;
+      }
+      break;
+
+    default:
+      as_fatal (_("internal inconsistency problem in %s:  %lx"),
+               __FUNCTION__, (long) fragP->fr_subtype);
+      break;
+    }
+
+  /* Now apply fix.  */
+  fix_new (fragP, fragP->fr_fix, 2, fragP->fr_symbol,
+          fragP->fr_offset, TRUE, rela);
+  /* Just fixed 2 bytes.  */
+  fragP->fr_fix += 2;
 }
 
+/* Relax fragment. Mostly stolen from hc11 and mcore
+   which arches I think I know.  */
 
-int
-md_estimate_size_before_relax (fragp, seg)
-     fragS *fragp ATTRIBUTE_UNUSED;
-     asection *seg ATTRIBUTE_UNUSED;
+long
+msp430_relax_frag (segT seg ATTRIBUTE_UNUSED, fragS * fragP,
+                  long stretch ATTRIBUTE_UNUSED)
 {
-  abort ();
-  return 0;
+  long growth;
+  offsetT aim = 0;
+  symbolS *symbolP;
+  const relax_typeS *this_type;
+  const relax_typeS *start_type;
+  relax_substateT next_state;
+  relax_substateT this_state;
+  const relax_typeS *table = md_relax_table;
+
+  /* Nothing to be done if the frag has already max size.  */
+  if (RELAX_STATE (fragP->fr_subtype) == STATE_UNDEF
+      || RELAX_STATE (fragP->fr_subtype) == STATE_WORD)
+    return 0;
+
+  if (RELAX_STATE (fragP->fr_subtype) == STATE_BITS10)
+    {
+      symbolP = fragP->fr_symbol;
+      if (symbol_resolved_p (symbolP))
+       as_fatal (_("internal inconsistency problem in %s: resolved symbol"),
+                 __FUNCTION__);
+      /* We know the offset. calculate a distance.  */
+      aim = S_GET_VALUE (symbolP) - fragP->fr_address - fragP->fr_fix;
+    }
+
+  this_state = fragP->fr_subtype;
+  start_type = this_type = table + this_state;
+
+  if (aim < 0)
+    {
+      /* Look backwards.  */
+      for (next_state = this_type->rlx_more; next_state;)
+       if (aim >= this_type->rlx_backward)
+         next_state = 0;
+       else
+         {
+           /* Grow to next state.  */
+           this_state = next_state;
+           this_type = table + this_state;
+           next_state = this_type->rlx_more;
+         }
+    }
+  else
+    {
+      /* Look forwards.  */
+      for (next_state = this_type->rlx_more; next_state;)
+       if (aim <= this_type->rlx_forward)
+         next_state = 0;
+       else
+         {
+           /* Grow to next state.  */
+           this_state = next_state;
+           this_type = table + this_state;
+           next_state = this_type->rlx_more;
+         }
+    }
+
+  growth = this_type->rlx_length - start_type->rlx_length;
+  if (growth != 0)
+    fragP->fr_subtype = this_state;
+  return growth;
 }
index e540124bc82c4aad3ba7cc96586e2893ee0766e8..c86e230965d0bd7d07eb7bf622c0b4123b9c495f 100644 (file)
@@ -1,5 +1,5 @@
 /* This file is tc-msp430.h
-   Copyright (C) 2002 Free Software Foundation, Inc.
+   Copyright (C) 2002, 2004 Free Software Foundation, Inc.
 
    Contributed by Dmitry Diky <diwil@mail.ru>
 
@@ -93,7 +93,7 @@
      of a PC relative instruction is the next instruction, so this
      macro would return the length of an instruction.  */
 
-extern long md_pcrel_from_section PARAMS ((struct fix *, segT));
+extern long md_pcrel_from_section (struct fix *, segT);
 
 #define LISTING_WORD_SIZE 2
 /*   The number of bytes to put into a word in a listing.  This affects
@@ -112,3 +112,7 @@ extern long md_pcrel_from_section PARAMS ((struct fix *, segT));
      should do nothing.  Some targets define a `.bss' directive that is
      also affected by this macro.  The default definition will set
      P2VAR to the truncated power of two of sizes up to eight bytes.  */
+
+#define md_relax_frag(SEG, FRAGP, STRETCH)             \
+   msp430_relax_frag (SEG, FRAGP, STRETCH)
+extern long msp430_relax_frag (segT, fragS *, long);
index 00503598538d6cdc22d6e02fde9f71e5a927aaeb..b398952280919560c2cff18bbe589e4130b30d29 100644 (file)
@@ -1,4 +1,4 @@
-@c Copyright 2002 Free Software Foundation, Inc.
+@c Copyright 2002, 2004 Free Software Foundation, Inc.
 @c This is part of the GAS manual.
 @c For copying conditions, see the file as.texinfo.
 @ifset GENERIC
@@ -19,6 +19,7 @@
 * MSP430 Floating Point::       Floating Point
 * MSP430 Directives::           MSP 430 Machine Directives
 * MSP430 Opcodes::              Opcodes
+* MSP430 Profiling Capability::        Profiling Capability
 @end menu
 
 @node MSP430 Options
@@ -116,6 +117,54 @@ Skips next N bytes followed by jump instruction and equivalent to
 
 @end table
 
+Also, there are some instructions, which cannot be found in other assemblers.
+These are branch instructions, which has different opcodes upon jump distance.
+They all got PC relative addressing mode.
+
+@table @code
+@item  beq label
+A polymorph instruction which is @samp{jeq label} in case if jump distance
+within allowed range for cpu's jump instruction. If not, this unrolls into
+a sequence of
+@smallexample
+  jne $+6
+  br  label
+@end smallexample
+
+@item bne label
+A polymorph instruction which is @samp{jne label} or @samp{jeq +4; br label}
+
+@item blt label
+A polymorph instruction which is @samp{jl label} or @samp{jge +4; br label}
+
+@item bltn label
+A polymorph instruction which is @samp{jn label} or @samp{jn +2; jmp +4; br label}
+
+@item bltu label
+A polymorph instruction which is @samp{jlo label} or @samp{jhs +2; br label}
+
+@item bge label
+A polymorph instruction which is @samp{jge label} or @samp{jl +4; br label}
+
+@item bgeu label
+A polymorph instruction which is @samp{jhs label} or @samp{jlo +4; br label}
+
+@item bgt label
+A polymorph instruction which is @samp{jeq +2; jge label} or @samp{jeq +6; jl  +4; br label}
+
+@item bgtu label
+A polymorph instruction which is @samp{jeq +2; jhs label} or @samp{jeq +6; jlo +4; br label}
+
+@item bleu label
+A polymorph instruction which is @samp{jeq label; jlo label} or @samp{jeq +2; jhs +4; br label}
+
+@item ble label
+A polymorph instruction which is @samp{jeq label; jl  label} or @samp{jeq +2; jge +4; br label}
+
+@item jump label
+A polymorph instruction which is @samp{jmp label} or @samp{br label}
+@end table
+
 
 @node MSP430 Floating Point
 @section Floating Point
@@ -150,6 +199,10 @@ MSP 430 assemblers.
 Currently this directive is ignored; it is accepted for compatibility with other
 MSP 430 assemblers.
 
+@cindex @code{profiler} directive, MSP 430
+@item .profiler
+This directive instructs assembler to add new profile entry to the object file.
+
 @end table
 
 @node MSP430 Opcodes
@@ -162,3 +215,98 @@ additional pseudo-instructions are needed on this family.
 
 For information on the 430 machine instruction set, see @cite{MSP430
 User's Manual, document slau049b}, Texas Instrument, Inc.
+
+@node MSP430 Profiling Capability
+@section Profiling Capability
+
+@cindex MSP 430 profiling capability
+@cindex profiling capability for MSP 430
+It is a performance hit to use gcc's profiling approach for this tiny target.
+Even more -- jtag hardware facility does not perform any profiling functions.
+However we've got gdb's built-in simulator where we can do anything.
+
+We define new section @samp{.profiler} which holds all profiling information.
+We define new pseudo operation @samp{.profiler} which will instruct assembler to
+add new profile entry to the object file. Profile should take place at the
+present address.
+
+Pseudo operation format:
+
+@samp{.profiler flags,function_to_profile [, cycle_corrector, extra]}
+
+
+where:
+
+@table @code
+
+@table @code
+
+@samp{flags} is a combination of the following characters:
+
+@item  s 
+function entry
+@item  x  
+function exit
+@item  i  
+function is in init section
+@item  f  
+function is in fini section
+@item  l  
+library call
+@item  c  
+libc standard call
+@item  d  
+stack value demand
+@item  I  
+interrupt service routine
+@item  P  
+prologue start
+@item  p  
+prologue end
+@item  E  
+epilogue start
+@item  e  
+epilogue end
+@item  j  
+long jump / sjlj unwind
+@item  a  
+an arbitrary code fragment
+@item t
+extra parameter saved (a constant value like frame size)
+@end table
+
+@item function_to_profile 
+a function address
+@item cycle_corrector  
+a value which should be added to the cycle counter, zero if omitted.
+@item extra  
+any extra parameter, zero if omitted.
+
+@end table
+
+For example:
+@smallexample
+.global fxx
+.type fxx,@@function
+fxx:
+.LFrameOffset_fxx=0x08
+.profiler "scdP", fxx     ; function entry.
+                         ; we also demand stack value to be saved
+  push r11
+  push r10
+  push r9
+  push r8
+.profiler "cdpt",fxx,0, .LFrameOffset_fxx  ; check stack value at this point
+                                         ; (this is a prologue end)
+                                         ; note, that spare var filled with 
+                                         ; the farme size
+  mov r15,r8
+...
+.profiler cdE,fxx         ; check stack
+  pop r8
+  pop r9
+  pop r10
+  pop r11
+.profiler xcde,fxx,3      ; exit adds 3 to the cycle counter
+  ret                     ; cause 'ret' insn takes 3 cycles
+@end smallexample
index d32f51e106f9c5b531c41043bff5b516bb1a5ed4..6b0cf0d4e0d213b567cde41b4b64a468eb6394af 100644 (file)
@@ -1,3 +1,7 @@
+2004-08-25  Dmitry Diky  <diwil@spec.ru>
+
+       * msp430.h: Add new relocs.
+
 2004-08-12  H.J. Lu  <hongjiu.lu@intel.com>
 
        * i386.h (R_386_USED_BY_INTEL_200): New.
index 912ded7685d37dc7c4e43ba430f2344c15b20031..2bf95a629b28d04d0554a1e7530d188329bf7c92 100644 (file)
@@ -1,5 +1,5 @@
 /* MSP430 ELF support for BFD.
-   Copyright (C) 2002 Free Software Foundation, Inc.
+   Copyright (C) 2002, 2004 Free Software Foundation, Inc.
    Contributed by Dmitry Diky <diwil@mail.ru>
 
    This file is part of BFD, the Binary File Descriptor library.
@@ -50,6 +50,8 @@ START_RELOC_NUMBERS (elf_msp430_reloc_type)
      RELOC_NUMBER (R_MSP430_16_PCREL,          4)
      RELOC_NUMBER (R_MSP430_16_BYTE,           5)
      RELOC_NUMBER (R_MSP430_16_PCREL_BYTE,     6)
+     RELOC_NUMBER (R_MSP430_2X_PCREL,          7)
+     RELOC_NUMBER (R_MSP430_RL_PCREL,          8)
 
 END_RELOC_NUMBERS (R_MSP430_max)
 
index b8ca48603631879659bd7574d37ed45feefe0a1a..6adb8fed99a1bdba158e467a388a6a75c1fff0e5 100644 (file)
@@ -1,3 +1,9 @@
+2004-08-24  Dmitry Diky  <diwil@spec.ru>
+
+       * msp430.h (msp430_opc): Add new instructions.
+       (msp430_rcodes): Declare new instructions.
+       (msp430_hcodes): Likewise..
+
 2004-08-13  Nick Clifton  <nickc@redhat.com>
 
        PR/301
index 19702254eefcfd65c254e5e936393aeb4c72db15..a59ede21517ea036b7cebc1c0e46daf1c8e88dcb 100644 (file)
@@ -1,6 +1,6 @@
 /* Opcode table for the TI MSP430 microcontrollers
 
-   Copyright 2002 Free Software Foundation, Inc.
+   Copyright 2002, 2004 Free Software Foundation, Inc.
    Contributed by Dmitry Diky <diwil@mail.ru>
    
    This program is free software; you can redistribute it and/or modify
@@ -103,9 +103,114 @@ static struct msp430_opcode_s msp430_opcodes[] =
   MSP_INSN (rra,   2, 1, 0x1100, 0xff80),
   MSP_INSN (swpb,  2, 1, 0x1080, 0xffc0),
   MSP_INSN (rrc,   2, 1, 0x1000, 0xff80),
+  /* Simple polymorphs.  */
+  MSP_INSN (beq,   4, 0, 0, 0xffff),
+  MSP_INSN (bne,   4, 1, 0, 0xffff),
+  MSP_INSN (blt,   4, 2, 0, 0xffff),
+  MSP_INSN (bltu,  4, 3, 0, 0xffff),
+  MSP_INSN (bge,   4, 4, 0, 0xffff),
+  MSP_INSN (bgeu,  4, 5, 0, 0xffff),
+  MSP_INSN (bltn,  4, 6, 0, 0xffff),
+  MSP_INSN (jump,  4, 7, 0, 0xffff),
+  /* Long polymorphs.  */
+  MSP_INSN (bgt,   5, 0, 0, 0xffff),
+  MSP_INSN (bgtu,  5, 1, 0, 0xffff),
+  MSP_INSN (bleu,  5, 2, 0, 0xffff),
+  MSP_INSN (ble,   5, 3, 0, 0xffff),
 
   /* End of instruction set.  */
   { NULL, 0, 0, 0, 0 }
 };
 
+/* GCC uses the some condition codes which we'll
+   implement as new polymorph instructions.
+  
+   COND        EXPL       SHORT JUMP   LONG JUMP
+   ===============================================
+   eq  ==         jeq          jne +4; br lab
+   ne  !=         jne          jeq +4; br lab
+
+   ltn honours no-overflow flag
+   ltn <          jn           jn +2;  jmp +4; br lab
+
+   lt  <          jl           jge +4; br lab 
+   ltu <          jlo          lhs +4; br lab
+   le  <= see below
+   leu <= see below
+
+   gt  >  see below
+   gtu >  see below
+   ge  >=         jge          jl +4; br lab
+   geu >=         jhs          jlo +4; br lab
+   ===============================================
+
+   Therefore, new opcodes are (BranchEQ -> beq; and so on...)
+   beq,bne,blt,bltn,bltu,bge,bgeu
+   'u' means unsigned compares 
+  
+   Also, we add 'jump' instruction:
+   jump        UNCOND  -> jmp          br lab
+
+   They will have fmt == 4, and insn_opnumb == number of instruction.  */
+
+struct rcodes_s 
+{
+  char * name;
+  int    index;        /* Corresponding insn_opnumb.  */
+  int    sop;  /* Opcode if jump length is short.  */
+  long   lpos; /* Label position.  */
+  long   lop0; /* Opcode 1 _word_ (16 bits).  */
+  long   lop1; /* Opcode second word.  */
+  long   lop2; /* Opcode third word.  */
+};
+
+#define MSP430_RLC(n,i,sop,o1) \
+  {#n, i, sop, 2, (o1 + 2), 0x4010, 0}
+
+static struct rcodes_s msp430_rcodes[] = 
+{
+  MSP430_RLC (beq,  0, 0x2400, 0x2000),
+  MSP430_RLC (bne,  1, 0x2000, 0x2400),
+  MSP430_RLC (blt,  2, 0x3800, 0x3400),
+  MSP430_RLC (bltu, 3, 0x2800, 0x2c00),
+  MSP430_RLC (bge,  4, 0x3400, 0x3800),
+  MSP430_RLC (bgeu, 5, 0x2c00, 0x2800),
+  {"bltn",          6, 0x3000, 3, 0x3000 + 1, 0x3c00 + 2,0x4010},
+  {"jump",          7, 0x3c00, 1, 0x4010, 0, 0},
+  {0,0,0,0,0,0,0}
+};
+#undef MSP430_RLC
+
+
+/* More difficult than above and they have format 5.
+   
+   COND        EXPL    SHORT                   LONG
+   =================================================================
+   gt  >       jeq +2; jge label       jeq +6; jl  +4; br label
+   gtu >       jeq +2; jhs label       jeq +6; jlo +4; br label
+   leu <=      jeq label; jlo label    jeq +2; jhs +4; br label
+   le  <=      jeq label; jl  label    jeq +2; jge +4; br label
+   =================================================================  */
+
+struct hcodes_s 
+{
+  char * name; 
+  int    index;                /* Corresponding insn_opnumb.  */
+  int    tlab;         /* Number of labels in short mode.  */
+  int    op0;          /* Opcode for first word of short jump.  */
+  int    op1;          /* Opcode for second word of short jump.  */
+  int    lop0;         /* Opcodes for long jump mode.  */
+  int    lop1;
+  int    lop2;
+};
+
+static struct hcodes_s msp430_hcodes[] = 
+{
+  {"bgt",  0, 1, 0x2401, 0x3400, 0x2403, 0x3802, 0x4010 },
+  {"bgtu", 1, 1, 0x2401, 0x2c00, 0x2403, 0x2802, 0x4010 },
+  {"bleu", 2, 2, 0x2400, 0x2800, 0x2401, 0x2c02, 0x4010 },
+  {"ble",  3, 2, 0x2400, 0x3800, 0x2401, 0x3402, 0x4010 },
+  {0,0,0,0,0,0,0,0}
+};
+
 #endif
index 9ca82f612f6efa46c51f60e8b204307bfcf0f357..413deace7ef3c81737c2535b435259b90007887b 100644 (file)
@@ -1,3 +1,8 @@
+2004-08-25  Dmitry Diky  <diwil@spec.ru>
+
+       * emulparams/msp430all.sh: Fix RAM sizes for all targets.
+       * scripttempl/elf32msp430.sc: Add .profiler section definition.
+
 2004-08-24  H.J. Lu  <hongjiu.lu@intel.com>
 
        * ldlang.c (wildcardp): Defined as a macro with strpbrk.
index e2ecc0d84f27163c8fe541f5925689039b2921d1..f6f2b4656170659c85ea94879b28d79be0294e2f 100644 (file)
@@ -288,7 +288,7 @@ TEMPLATE_NAME=generic
 ROM_START=0x4000
 ROM_SIZE=0xbfe0
 RAM_START=0x0200
-RAM_SIZE=0x07ff
+RAM_SIZE=0x0800
 
 STACK=0xa00
 fi
@@ -305,7 +305,7 @@ TEMPLATE_NAME=generic
 ROM_START=0x1100
 ROM_SIZE=0xeee0
 RAM_START=0x0200
-RAM_SIZE=0x07ff
+RAM_SIZE=0x0800
 
 STACK=0xa00
 fi
@@ -390,7 +390,7 @@ TEMPLATE_NAME=generic
 ROM_START=0x4000
 ROM_SIZE=0xbfe0
 RAM_START=0x0200
-RAM_SIZE=0x07ff
+RAM_SIZE=0x0800
 
 STACK=0xa00
 fi
@@ -407,7 +407,7 @@ TEMPLATE_NAME=generic
 ROM_START=0x1100
 ROM_SIZE=0xeee0
 RAM_START=0x0200
-RAM_SIZE=0x07ff
+RAM_SIZE=0x0800
 
 STACK=0xa00
 fi
@@ -424,9 +424,9 @@ TEMPLATE_NAME=generic
 ROM_START=0x8000
 ROM_SIZE=0x7fe0
 RAM_START=0x1100
-RAM_SIZE=0x13ff
+RAM_SIZE=0x1400
 
-STACK=0x1400
+STACK=0x2500
 fi
 
 if [ "${MSP430_NAME}" = "msp430x1611" ] ; then
@@ -441,9 +441,9 @@ TEMPLATE_NAME=generic
 ROM_START=0x4000
 ROM_SIZE=0xbfe0
 RAM_START=0x1100
-RAM_SIZE=0x27ff
+RAM_SIZE=0x2800
 
-STACK=0x2800
+STACK=0x3900
 fi
 
 if [ "${MSP430_NAME}" = "msp430x1612" ] ; then
@@ -455,12 +455,12 @@ MAXPAGESIZE=1
 EMBEDDED=yes
 TEMPLATE_NAME=generic
 
-ROM_START=0x2800
-ROM_SIZE=0xd7e0
+ROM_START=0x2500
+ROM_SIZE=0xdae0
 RAM_START=0x1100
-RAM_SIZE=0x13ff
+RAM_SIZE=0x1400
 
-STACK=0x1400
+STACK=0x2500
 fi
 
 if [ "${MSP430_NAME}" = "msp430x311" ] ; then
@@ -764,7 +764,7 @@ TEMPLATE_NAME=generic
 ROM_START=0x4000
 ROM_SIZE=0xbfe0
 RAM_START=0x0200
-RAM_SIZE=0x07ff
+RAM_SIZE=0x0800
 
 STACK=0xa00
 fi
@@ -781,7 +781,7 @@ TEMPLATE_NAME=generic
 ROM_START=0x1100
 ROM_SIZE=0xeee0
 RAM_START=0x0200
-RAM_SIZE=0x07ff
+RAM_SIZE=0x0800
 
 STACK=0xa00
 fi
@@ -866,7 +866,7 @@ TEMPLATE_NAME=generic
 ROM_START=0x4000
 ROM_SIZE=0xbef0
 RAM_START=0x0200
-RAM_SIZE=0x07ff
+RAM_SIZE=0x0800
 
 STACK=0xa00
 fi
@@ -883,7 +883,7 @@ TEMPLATE_NAME=generic
 ROM_START=0x1100
 ROM_SIZE=0xeee0
 RAM_START=0x0200
-RAM_SIZE=0x07ff
+RAM_SIZE=0x0800
 
 STACK=0xa00
 fi
@@ -934,7 +934,7 @@ TEMPLATE_NAME=generic
 ROM_START=0x8000
 ROM_SIZE=0x7fe0
 RAM_START=0x0200
-RAM_SIZE=1K
+RAM_SIZE=0x400
 
 STACK=0x600
 fi
index 5754d662dcf62b52ec1c2e9b28b95b05b10aecd2..cbffe48221956414de9bcfd455ec95e8793e45ab 100644 (file)
@@ -202,6 +202,9 @@ SECTIONS
 
   ${HEAP_SECTION_MSP430}
 
+  /* Stabs for profiling information*/
+  .profiler 0 : { *(.profiler) }
+  
   /* Stabs debugging sections.  */
   .stab 0 : { *(.stab) } 
   .stabstr 0 : { *(.stabstr) }