ARM ELF: ensure that the tag types are corrected

Ensure that the tag types are reflected on a replacement.  This is particularly
important for the compatibility tag which has multiple representations where the
last definition wins.

llvm-svn: 199577

Author: compnerd

llvm/lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp

@@ -290,6 +290,7 @@ class ARMTargetELFStreamer : public ARMTargetStreamer {
     if (AttributeItem *Item = getAttributeItem(Attribute)) {
       if (!OverwriteExisting)
         return;
+      Item->Type = AttributeItem::NumericAttribute;
       Item->IntValue = Value;
       return;
     }
@@ -310,6 +311,7 @@ class ARMTargetELFStreamer : public ARMTargetStreamer {
     if (AttributeItem *Item = getAttributeItem(Attribute)) {
       if (!OverwriteExisting)
         return;
+      Item->Type = AttributeItem::TextAttribute;
       Item->StringValue = Value;
       return;
     }
@@ -330,6 +332,7 @@ class ARMTargetELFStreamer : public ARMTargetStreamer {
     if (AttributeItem *Item = getAttributeItem(Attribute)) {
       if (!OverwriteExisting)
         return;
+      Item->Type = AttributeItem::NumericAndTextAttributes;
       Item->IntValue = IntValue;
       Item->StringValue = StringValue;
       return;

llvm/test/MC/ARM/directive-eabi_attribute-overwrite.s

@@ -0,0 +1,18 @@
+@ RUN: llvm-mc -triple armv7-eabi -filetype obj -o - %s | llvm-readobj -s -sd \
+@ RUN:   | FileCheck %s
+
+	.syntax unified
+	.thumb
+
+	.eabi_attribute Tag_compatibility, 1
+	.eabi_attribute Tag_compatibility, 1, "aeabi"
+
+@ CHECK: Section {
+@ CHECK:   Name: .ARM.attributes
+@ CHECK:   Type: SHT_ARM_ATTRIBUTES
+@ CHECK:   SectionData (
+@ CHECK:     0000: 41170000 00616561 62690001 0D000000
+@ CHECK:     0010: 20014145 41424900
+@ CHECK:   )
+@ CHECK: }
+