As Aaron pointed out it's simpler to reject wide string availability attr messages in the parser.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@190706 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Sema/SemaDeclAttr.cpp b/lib/Sema/SemaDeclAttr.cpp
index 8d01d63..8530c8b 100644
--- a/lib/Sema/SemaDeclAttr.cpp
+++ b/lib/Sema/SemaDeclAttr.cpp
@@ -2267,16 +2267,9 @@
AvailabilityChange Obsoleted = Attr.getAvailabilityObsoleted();
bool IsUnavailable = Attr.getUnavailableLoc().isValid();
StringRef Str;
- if (Attr.getMessageExpr()) {
- const StringLiteral *SE = dyn_cast<StringLiteral>(Attr.getMessageExpr());
- if (!SE || !SE->isAscii()) {
- S.Diag(Attr.getMessageExpr()->getLocStart(),
- diag::err_attribute_argument_type)
- << Attr.getName() << AANT_ArgumentString;
- return;
- }
+ if (const StringLiteral *SE =
+ dyn_cast_or_null<StringLiteral>(Attr.getMessageExpr()))
Str = SE->getString();
- }
AvailabilityAttr *NewAttr = S.mergeAvailabilityAttr(ND, Attr.getRange(), II,
Introduced.Version,