Re: r246830 - Fix a potential APInt memory leak when using __attribute__((flag_enum)), and

In case you didn’t get an email from the failure because it was overshadowed by the previous error. This commit seems to break the green dragon bots: http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA-incremental_check/12990/testReport/junit/Clang/Sema/attr_flag_enum_c/

[PATCH] D12622: [Shave]: add a -MT option to moviCompile if there wasn't one

dougk created this revision. dougk added a reviewer: chandlerc. dougk added a subscriber: cfe-commits. http://reviews.llvm.org/D12622 Files: lib/Driver/Tools.cpp test/Driver/shave-toolchain.c Index: test/Driver/shave-toolchain.c ===

Re: [PATCH] D12619: [Static Analyzer] Minor cleanups for the nullability checker.

xazax.hun added a comment. They are in the anonymous namespace. Maybe it would be cleaner to move them outside of the namespace and preserve the static keyword? http://reviews.llvm.org/D12619 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D12624: Top-level anonymous namespaces are missing import DW_TAG_imported_module and nested anonymous namespaces are not

kromanova created this revision. kromanova added reviewers: rsmith, dblaikie, echristo, probinson, aprantl. kromanova added a subscriber: cfe-commits. I had a review opened for anonymous namespaces bug in llvm-commits (see http://reviews.llvm.org/D7895). It was opened in llvm-commits is because

Re: r246830 - Fix a potential APInt memory leak when using __attribute__((flag_enum)), and

On Thu, Sep 3, 2015 at 6:21 PM, Steven Wu via cfe-commits < cfe-commits@lists.llvm.org> wrote: > In case you didn’t get an email from the failure because it was > overshadowed by the previous error. This commit seems to break the green > dragon bots: > > http://lab.llvm.org:8080/green/job/clang-st

Re: [PATCH] D12594: [X86-64] Allow additional register names in inline assembler.

ABataev added a comment. Eric, thanks for the review! Will do! http://reviews.llvm.org/D12594 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r246835 - [X86-64] Allow additional register names in inline assembler.

Author: abataev Date: Thu Sep 3 22:42:23 2015 New Revision: 246835 URL: http://llvm.org/viewvc/llvm-project?rev=246835&view=rev Log: [X86-64] Allow additional register names in inline assembler. Patch allows to recognize additional registers x8d, x8b, x8w - x15d, x15b, x15w in inline assembler,

Re: [PATCH] D12594: [X86-64] Allow additional register names in inline assembler.

This revision was automatically updated to reflect the committed changes. Closed by commit rL246835: [X86-64] Allow additional register names in inline assembler. (authored by ABataev). Changed prior to commit: http://reviews.llvm.org/D12594?vs=33926&id=34011#toc Repository: rL LLVM http://

Re: [PATCH] D1623: Support __builtin_ms_va_list.

cdavis5x updated this revision to Diff 34012. cdavis5x added a comment. Address @rsmith's comments. http://reviews.llvm.org/D1623 Files: include/clang/AST/ASTContext.h include/clang/AST/Expr.h include/clang/Basic/BuiltinsX86.def include/clang/Basic/DiagnosticSemaKinds.td include/clang

Re: [PATCH] D1623: Support __builtin_ms_va_list.

cdavis5x added inline comments. Comment at: include/clang/AST/Expr.h:3709 @@ -3708,3 +3708,3 @@ VAArgExpr(SourceLocation BLoc, Expr* e, TypeSourceInfo *TInfo, -SourceLocation RPLoc, QualType t) +SourceLocation RPLoc, QualType t, bool IsMS = false) :

Re: r246830 - Fix a potential APInt memory leak when using __attribute__((flag_enum)), and

OK, should be fixed in r246836. On Thu, Sep 3, 2015 at 7:50 PM, Richard Smith wrote: > On Thu, Sep 3, 2015 at 6:21 PM, Steven Wu via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> In case you didn’t get an email from the failure because it was >> overshadowed by the previous error. This

r246837 - Cleanups, no functionality change.

Author: rsmith Date: Thu Sep 3 23:09:21 2015 New Revision: 246837 URL: http://llvm.org/viewvc/llvm-project?rev=246837&view=rev Log: Cleanups, no functionality change. Modified: cfe/trunk/lib/Sema/SemaDecl.cpp Modified: cfe/trunk/lib/Sema/SemaDecl.cpp URL: http://llvm.org/viewvc/llvm-projec

Re: r246830 - Fix a potential APInt memory leak when using __attribute__((flag_enum)), and

Sorry I just get back to my computer. I will watch the bots going. I will try investigating if it is still failing. As far as I know, there is no special settingl about the bots and all our bots are complaining about it. Steven > On Sep 3, 2015, at 9:09 PM, Richard Smith wrote: > > OK, shoul

Re: r246830 - Fix a potential APInt memory leak when using __attribute__((flag_enum)), and

Looks fixed. Thank a lot for investigating! > On Sep 3, 2015, at 9:21 PM, Steven Wu via cfe-commits > wrote: > > Sorry I just get back to my computer. I will watch the bots going. I will try > investigating if it is still failing. > As far as I know, there is no special settingl about the bots

Re: [PATCH] D12619: [Static Analyzer] Minor cleanups for the nullability checker.

zaks.anna added inline comments. Comment at: lib/StaticAnalyzer/Checkers/NullabilityChecker.cpp:99 @@ +98,3 @@ + "Nullable pointer is assigned to a " + "pointer which has _Nonnull type", +

[PATCH] D12627: Fix a small bug in clang where generating some temporary files would have an extra period before the extension.

dirty created this revision. dirty added a reviewer: akyrtzi. dirty added a subscriber: cfe-commits. Fix a small bug in clang where generating some temporary files would have an extra period before the extension. http://reviews.llvm.org/D12627 Files: lib/ARCMigrate/FileRemapper.cpp

r246838 - Fix a couple of \param(s) in r246815. [-Wdocumentation]

Author: chapuni Date: Fri Sep 4 00:19:31 2015 New Revision: 246838 URL: http://llvm.org/viewvc/llvm-project?rev=246838&view=rev Log: Fix a couple of \param(s) in r246815. [-Wdocumentation] Modified: cfe/trunk/lib/CodeGen/CGClass.cpp Modified: cfe/trunk/lib/CodeGen/CGClass.cpp URL: http://l

Re: [PATCH] D12471: Correct documentation for numSelectorArgs matcher

kastiglione added a comment. Thanks @klimek! There's certainly no rush on a diff like this :P http://reviews.llvm.org/D12471 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

<    1   2