<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>On Mar 21, 2013, at 11:08 AM, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:</div><div><br class="Apple-interchange-newline"><blockquote type="cite"><p dir="ltr">Delayed template parsing is not supported by the ast serialization system yet. Maybe use-fno-delayed-template-parsing or run cc1 directly instead of the driver?</p><div><br></div></blockquote><div><br></div><div>Aaron, could you try Richard's suggestion, I don't have a windows machine to test.</div><br><br><blockquote type="cite">
<div class="gmail_quote">On 21 Mar 2013 09:17, "Aaron Ballman" <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I am seeing the same breakage as Guy, and it's turning the build bots<br>
red. Can you look into this?<br>
<br>
Thanks!<br>
<br>
~Aaron<br>
<br>
On Tue, Mar 19, 2013 at 9:15 AM, Benyei, Guy <<a href="mailto:guy.benyei@intel.com">guy.benyei@intel.com</a>> wrote:<br>
> Hi Argyrios,<br>
> The test you've added here fails when I build it on Windows. For some reason, when compiling C++ to AST and then compiling AST to llvm, the S<T>::mf function is declared but not defined in the resulting module. It seems to me a buggy behavior.<br>
><br>
> Any insights on this issue?<br>
><br>
> Thanks<br>
> Guy<br>
><br>
> -----Original Message-----<br>
> From: <a href="mailto:cfe-commits-bounces@cs.uiuc.edu">cfe-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:cfe-commits-bounces@cs.uiuc.edu">cfe-commits-bounces@cs.uiuc.edu</a>] On Behalf Of Argyrios Kyrtzidis<br>
> Sent: Tuesday, March 19, 2013 00:55<br>
> To: <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> Subject: r177336 - [frontend] Initialize the diagnostic client before loading an ast file.<br>
><br>
> Author: akirtzidis<br>
> Date: Mon Mar 18 17:55:24 2013<br>
> New Revision: 177336<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=177336&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=177336&view=rev</a><br>
> Log:<br>
> [frontend] Initialize the diagnostic client before loading an ast file.<br>
><br>
> Issue reported by Tom Honermann!<br>
> <a href="http://llvm.org/bugs/show_bug.cgi?id=15377" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=15377</a><br>
><br>
> Added:<br>
> cfe/trunk/test/Frontend/ast-main.cpp<br>
> Modified:<br>
> cfe/trunk/lib/Frontend/FrontendAction.cpp<br>
><br>
> Modified: cfe/trunk/lib/Frontend/FrontendAction.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/FrontendAction.cpp?rev=177336&r1=177335&r2=177336&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/FrontendAction.cpp?rev=177336&r1=177335&r2=177336&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/lib/Frontend/FrontendAction.cpp (original)<br>
> +++ cfe/trunk/lib/Frontend/FrontendAction.cpp Mon Mar 18 17:55:24 2013<br>
> @@ -188,6 +188,10 @@ bool FrontendAction::BeginSourceFile(Com<br>
><br>
> setCurrentInput(Input, AST);<br>
><br>
> + // Inform the diagnostic client we are processing a source file.<br>
> + CI.getDiagnosticClient().BeginSourceFile(CI.getLangOpts(), 0);<br>
> + HasBegunSourceFile = true;<br>
> +<br>
> // Set the shared objects, these are reset when we finish processing the<br>
> // file, otherwise the CompilerInstance will happily destroy them.<br>
> CI.setFileManager(&AST->getFileManager());<br>
><br>
> Added: cfe/trunk/test/Frontend/ast-main.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Frontend/ast-main.cpp?rev=177336&view=auto" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Frontend/ast-main.cpp?rev=177336&view=auto</a><br>
> ==============================================================================<br>
> --- cfe/trunk/test/Frontend/ast-main.cpp (added)<br>
> +++ cfe/trunk/test/Frontend/ast-main.cpp Mon Mar 18 17:55:24 2013<br>
> @@ -0,0 +1,22 @@<br>
> +// RUN: %clang -emit-llvm -S -o %t1.ll -x c++ - < %s // RUN: %clang<br>
> +-emit-ast -o %t.ast %s // RUN: %clang -emit-llvm -S -o %t2.ll -x ast -<br>
> +< %t.ast // RUN: diff %t1.ll %t2.ll<br>
> +<br>
> +// <a href="http://llvm.org/bugs/show_bug.cgi?id=15377" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=15377</a><br>
> +template<typename T><br>
> +struct S {<br>
> + T *mf();<br>
> +};<br>
> +template<typename T><br>
> +T *S<T>::mf() {<br>
> + // warning: control reaches end of non-void function<br>
> +[-Wreturn-type] }<br>
> +<br>
> +void f() {<br>
> + S<int>().mf();<br>
> +}<br>
> +<br>
> +int main() {<br>
> + return 0;<br>
> +}<br>
><br>
><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>
> Intel Israel (74) Limited<br>
><br>
> This e-mail and any attachments may contain confidential material for<br>
> the sole use of the intended recipient(s). Any review or distribution<br>
> by others is strictly prohibited. If you are not the intended<br>
> recipient, please contact the sender and delete all copies.<br>
><br>
><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>
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>
</blockquote></div>
_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits<br></blockquote></div><br></body></html>