[cfe-commits] r145244 - in /cfe/trunk: include/clang/AST/ExprCXX.h include/clang/Sema/Initialization.h include/clang/Sema/Sema.h lib/Sema/Sema.cpp lib/Sema/SemaCast.cpp lib/Sema/SemaExprCXX.cpp lib/Sema/SemaInit.cpp test/Index/preamble_macro_template.cpp test/Index/recursive-cxx-member-calls.cpp

[Nicola Gigante]

Il giorno 01/dic/2011, alle ore 20:15, Richard Smith ha scritto:

> On Thu, December 1, 2011 12:09, Nicola Gigante wrote:
>> Il giorno 30/nov/2011, alle ore 00:06, Richard Smith ha scritto:
>>> On Tue, November 29, 2011 22:28, Richard Smith wrote:
>>>> This change produces a broken AST for this testcase:
>>>> 
>>>> _Complex double d = 0.0;
>>>> int n = static_cast<int>(d);
>>>> 
>>>> Please fix or revert. Thanks!
>>>> Richard
>> 
>> I'm sorry, this was expected to work of course.
>> I'll look at it as soon as possible.
>> 
>>> This patch is also causing us to build broken ASTs when compiling Chromium,
>>> for what appears to be an unrelated reason. I've reverted the patch in
>>> r145447.
>> 
>> What do you mean by "broken AST" here?
> 
> We are creating a CXXStaticCast node with a null TypeAsWritten. I'm trying to
> obtain a reduced testcase for you…
> 

Thank you.
In the meantime, I've attached this revised patch that should fix the issue of
_Complex casts. Would you try it please?

I'm also working on a way to pass the SourceLocations and the TypeSourceInfo
to CastExprToType(), but I'm having some problems. I'll have
more news on this as soon as possible.


> Thanks,
> Richard
> 

Bye,
Nicola

-------------- next part --------------
A non-text attachment was scrubbed...
Name: cast.patch
Type: application/octet-stream
Size: 31858 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20111202/0c40dd07/attachment.obj>