Fix mapping symbol handling on arm 32/64

[tberghammer]

Summary

Fix mapping symbol handling on arm 32/64

The debug info section contains some $d mapping symbol what is
overlapping with code sections in other sections of the object file
causing problem in the address class detection. This CL ignores these
symbols from the address class map as the debug info sections don't use
this map.

source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp

for (i = 0; i < num_symbols; ++i)
         {
             if (arch.GetMachine() == llvm::Triple::arm)
             {
                 if (symbol.getBinding() == STB_LOCAL && symbol_name && symbol_name[0] == '$')
                 {
                     // These are reserved for the specification (e.g.: mapping
                     // symbols). We don't want to add them to the symbol table.
 
+                    if (symbol_type == eSymbolTypeCode)
+                    {
                         llvm::StringRef symbol_name_ref(symbol_name);
                         if (symbol_name_ref == "$a" || symbol_name_ref.startswith("$a."))
                         {
                             // $a[.<any>]* - marks an ARM instruction sequence
                             m_address_class_map[symbol.st_value] = eAddressClassCode;
                         }
                         else if (symbol_name_ref == "$b" || symbol_name_ref.startswith("$b.") ||
                                  symbol_name_ref == "$t" || symbol_name_ref.startswith("$t."))
                         {
                             // $b[.<any>]* - marks a THUMB BL instruction sequence
                             // $t[.<any>]* - marks a THUMB instruction sequence
                             m_address_class_map[symbol.st_value] = eAddressClassCodeAlternateISA;
                         }
                         else if (symbol_name_ref == "$d" || symbol_name_ref.startswith("$d."))
                         {
                             // $d[.<any>]* - marks a data item sequence (e.g. lit pool)
                             m_address_class_map[symbol.st_value] = eAddressClassData;
                         }
+                    }
 
                     continue;
                 }
             }
             else if (arch.GetMachine() == llvm::Triple::aarch64)
             {
                 if (symbol.getBinding() == STB_LOCAL && symbol_name && symbol_name[0] == '$')
                 {
                     // These are reserved for the specification (e.g.: mapping
                     // symbols). We don't want to add them to the symbol table.
 
+                    if (symbol_type == eSymbolTypeCode)
+                    {
                         llvm::StringRef symbol_name_ref(symbol_name);
                         if (symbol_name_ref == "$x" || symbol_name_ref.startswith("$x."))
                         {
                             // $x[.<any>]* - marks an A64 instruction sequence
                             m_address_class_map[symbol.st_value] = eAddressClassCode;
                         }
                         else if (symbol_name_ref == "$d" || symbol_name_ref.startswith("$d."))
                         {
                             // $d[.<any>]* - marks a data item sequence (e.g. lit pool)
                             m_address_class_map[symbol.st_value] = eAddressClassData;
                         }
+                    }
 
                     continue;
                 }
             }
 
             if (arch.GetMachine() == llvm::Triple::arm)
             {
                 // THUMB functions have the lower bit of their address set. Fixup