[PATCH] ARM IAS: improve .eabi_attribute handling

[Logan Chien]

================
Comment at: lib/Target/ARM/AsmParser/ARMAsmParser.cpp:8152
@@ -8151,3 +8151,3 @@
 /// parseDirectiveEabiAttr
 ///  ::= .eabi_attribute int, int
 bool ARMAsmParser::parseDirectiveEabiAttr(SMLoc L) {
----------------
It seems that the comment is out-of-date.  I think the following comments should be added as well:

    /// ::= .eabi_attribute int, int, "str"
    /// ::= .eabi_attribute Tag_name, int
    /// ::= .eabi_attribute Tag_name, int, "str"

================
Comment at: lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp:350
@@ -309,1 +349,3 @@
+  virtual void emitIntTextAttribute(unsigned Attribute, unsigned IntValue,
+                                    StringRef StringValue = "");
   virtual void emitArch(unsigned Arch);
----------------
I think we can drop the default value for "StringValue" argument.


http://llvm-reviews.chandlerc.com/D2484