<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 15, 2015 at 12:48 AM, Nicolas Brunie <span dir="ltr"><<a href="mailto:nicolas.brunie@kalray.eu" target="_blank">nicolas.brunie@kalray.eu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi All,<br>
I just submitted bug 22236 for clang (<a href="http://llvm.org/bugs/show_bug.cgi?id=22236" target="_blank">http://llvm.org/bugs/show_<u></u>bug.cgi?id=22236</a>) and was asked to submit my suggested commit to this list.<br>
So here it is. This is a very low priority, "error message bug", but since I would like to contribute to the project in the future, It seems a good introduction to start with that submission.</blockquote><div><br></div><div>Indeed, this is a great place to start. Committed in r226328. Could you close PR22236?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"> Sorry if I did not follow the protocol adequately.<br></blockquote><div><br></div><div>We generally tend to prefer having patches as attachments (which seems to be the only universal way to ensure all email clients to not mangle the patch), but this patch was so small I was able to apply it manually no problem.</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
Best regards,<br>
Nicolas Brunie<br>
<br>
P.S: while preparing a commit for clang trunk rather than the LLVM 3.5.1 version I am using, I found that an other message was wrong and included it in the patch<br>
<br>
>From f3ca32d45c67464221175f4367b97d<u></u>055aa377f8 Mon Sep 17 00:00:00 2001<br>
From: Nicolas Brunie <<a href="mailto:nicolas.brunie@kalray.eu" target="_blank">nicolas.brunie@kalray.eu</a>><br>
Date: Thu, 15 Jan 2015 09:41:55 +0100<br>
Subject: [PATCH] [#22236] fixing error messages in DecodeTypeFromStr<br>
<br>
---<br>
lib/AST/ASTContext.cpp | 4 ++--<br>
1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/lib/AST/ASTContext.cpp b/lib/AST/ASTContext.cpp<br>
index 6b864d0..68e594b 100644<br>
--- a/lib/AST/ASTContext.cpp<br>
+++ b/lib/AST/ASTContext.cpp<br>
@@ -7543,7 +7543,7 @@ static QualType DecodeTypeFromStr(const char *&Str, const ASTContext &Context,<br>
break;<br>
case 'U':<br>
assert(!Signed && "Can't use both 'S' and 'U' modifiers!");<br>
- assert(!Unsigned && "Can't use 'S' modifier multiple times!");<br>
+ assert(!Unsigned && "Can't use 'U' modifier multiple times!");<br>
Unsigned = true;<br>
break;<br>
case 'L':<br>
@@ -7578,7 +7578,7 @@ static QualType DecodeTypeFromStr(const char *&Str, const ASTContext &Context,<br>
break;<br>
case 'h':<br>
assert(HowLong == 0 && !Signed && !Unsigned &&<br>
- "Bad modifiers used with 'f'!");<br>
+ "Bad modifiers used with 'h'!");<br>
Type = Context.HalfTy;<br>
break;<br>
case 'f':<span class=""><font color="#888888"><br>
-- <br>
1.8.3.1<br>
<br>
</font></span><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div></div>