[Dwarf-discuss] Some remarks about dwarf.h

[Kees Bakker]

Hi,

Thanks for the update in dwarf.h
There are a few things I noticed.

1) DW_CFA_lo_user, DW_CFA_hi_user is spelled a bit different
   (DW_CFA_low_user and DW_CFA_high_user).

2) After a closer look at dwarf.h I noticed a conflict between
   DW_ATE_HP_float80 and DW_ATE_ALTIUM_fract and
   DW_ATE_HP_complex_float80 and DW_ATE_ALTIUM_accum
   Perhaps this should be notified with a "conflict" comment.

3) It would be better to put the lines with the extension
   values after the lo_user. Have a look at:

   ...
   /* HP extensions. */
   #define DW_AT_HP_block_index                    0x2000  /* HP */

   #define DW_AT_lo_user                           0x2000
   ...
    /* GNU extensions. */
   #define DW_OP_GNU_push_tls_address      0xe0 /* GNU */
   #define DW_OP_lo_user                   0xe0
   ...

   /* HP Floating point extensions. */
   #define DW_ATE_HP_float80             0x80 /* (80 bit). HP */

   #define DW_ATE_lo_user                  0x80

   /* HP Floating point extensions. */
   #define DW_ATE_HP_complex_float80     0x81 /* Complex (80 bit). HP  */
   ...
-- 
**************************************
Kees Bakker
Senior Software Designer
Altium - Think it, Design it, Build it
Phone  : +31 33 455 8584
E-Mail : Kees.Bakker at altium.nl
URL    : http://www.altium.com/
**************************************
Become a programmer and never see the world!!