r184048 - Updated the support for contextual conversion tweaks (n3323) with a previously overlooked part: implicitly converting array sizes to size_t, rather than contextually converting them to some unique type.

Renato Golin renato.golin at linaro.org
Mon Jun 17 01:38:32 PDT 2013


Hi Evgenly,

These errors are different than the ones we were seeing in 32-bits. The
latest patch did fix our bots.

I'm seeing 10 warnings on our bootstrap build, but not that one, which is
weird because it doesn't look like a 64-bit specific part of the code.

cheers,
--renato


On 17 June 2013 09:29, Evgeniy Stepanov <eugeni.stepanov at gmail.com> wrote:

> Seems like r184053 did not help.
> I see failures on 64-bit bootstrap with -Werror. This is not 32-bit
> specific.
>
> /code/llvm/tools/clang/lib/Sema/SemaCast.cpp:772:48: error: template
> argument uses local type '<anonymous enum at
> /code/llvm/tools/clang/lib/Sema/SemaCast.cpp:710:3>'
> [-Werror,-Wlocal-type-template-args]
>     << DerivedType << BaseType << !VirtualBase << ReinterpretKind
>                                                ^~
> /code/llvm/tools/clang/lib/Sema/AnalysisBasedWarnings.cpp:1126:7:
> error: template argument uses local type '<anonymous enum at
> /code/llvm/tools/clang/lib/Sema/AnalysisBasedWarnings.cpp:1105:5>'
> [-Werror,-Wlocal-type-template-args]
>       << ObjectKind << D << FunctionKind
>       ^~
> /code/llvm/tools/clang/lib/Sema/AnalysisBasedWarnings.cpp:1126:26:
> error: template argument uses local type '<anonymous enum at
> /code/llvm/tools/clang/lib/Sema/AnalysisBasedWarnings.cpp:1067:3>'
> [-Werror,-Wlocal-type-template-args]
>       << ObjectKind << D << FunctionKind
>                          ^~
>
> On Sun, Jun 16, 2013 at 7:02 PM, Larisse Voufo <lvoufo at google.com> wrote:
> >
> >
> >
> > On Sun, Jun 16, 2013 at 6:13 AM, Renato Golin <renato.golin at linaro.org>
> > wrote:
> >>
> >> On 16 June 2013 14:10, Renato Golin <renato.golin at linaro.org> wrote:
> >>>
> >>> Random failures occur, but they tend to be compilation errors, for
> >>> instance for lack of memory or timeout. Sometimes, MCJIT tests fail
> >>> randomly, and we're trying to get those fixed as well. So, if the
> compiler
> >>> error is something like "unrecoverable error" with no error message,
> it's
> >>> probably not your fault. If the error is in a test that has nothing to
> do
> >>> with your change (ex. MCJIT failures after a C++11 change), it should
> be
> >>> safe to ignore, too. But we appreciate
> >>
> >>
> >> Sorry, CTRL+ENTER sends the email on GMail and I hit it while trying
> >> CTRL+BACKSPC. Continuing...
> >>
> >> We appreciate if you could always have a look at the breaking bots when
> >> you receive an email from the buildmaster, as you could fix the bug
> while
> >> the maintainers are sleeping... ;)
> >
> >
> > Roger. Thanks for the info. I think the r184053 push fix some of these.
> I'm
> > verifying my tests execution in 32-bit mode.
> >
> >>
> >>
> >> cheers,
> >> --renato
> >
> >
> >
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130617/352fa559/attachment.html>


More information about the cfe-commits mailing list