[PATCH] D61909: Add Clang shared library with C++ exports

2019-05-18 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru added a comment. @beanz Great doc, thanks! I will see what I could use for Debian/Ubuntu packages (as we have a lot users and packages organized in a specific way, it isn't always easy to make huge changes) Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llv

r361096 - ASTNodeImporter - fix uninitialized variable warnings. NFCI.

2019-05-18 Thread Simon Pilgrim via cfe-commits
Author: rksimon Date: Sat May 18 04:33:27 2019 New Revision: 361096 URL: http://llvm.org/viewvc/llvm-project?rev=361096&view=rev Log: ASTNodeImporter - fix uninitialized variable warnings. NFCI. Modified: cfe/trunk/lib/AST/ASTImporter.cpp Modified: cfe/trunk/lib/AST/ASTImporter.cpp URL: htt

r361097 - MIGChecker - assert we have a non-null LocationContext. NFCI.

2019-05-18 Thread Simon Pilgrim via cfe-commits
Author: rksimon Date: Sat May 18 04:42:19 2019 New Revision: 361097 URL: http://llvm.org/viewvc/llvm-project?rev=361097&view=rev Log: MIGChecker - assert we have a non-null LocationContext. NFCI. Fixes scan-build warning. Modified: cfe/trunk/lib/StaticAnalyzer/Checkers/MIGChecker.cpp Modifi

[PATCH] D62093: [analyzer] Divide checkers into 3 categories: released, alpha, developer

2019-05-18 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. Let's come to a consensus on this, and followup patches, but as soon as that's done, this REALLY NEEDS to be documented in the release notes. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62093/new/ https://reviews.llvm.org/D62093 _

r361098 - ScalarExprEmitter::EmitCompoundAssign - fix uninitialized variable warning. NFCI.

2019-05-18 Thread Simon Pilgrim via cfe-commits
Author: rksimon Date: Sat May 18 05:17:15 2019 New Revision: 361098 URL: http://llvm.org/viewvc/llvm-project?rev=361098&view=rev Log: ScalarExprEmitter::EmitCompoundAssign - fix uninitialized variable warning. NFCI. Modified: cfe/trunk/lib/CodeGen/CGExprScalar.cpp Modified: cfe/trunk/lib/Co

r361099 - [analyzer] PR41753: Include complex integer types in NonLoc::isCompoundType

2019-05-18 Thread Kristof Umann via cfe-commits
Author: szelethus Date: Sat May 18 05:34:08 2019 New Revision: 361099 URL: http://llvm.org/viewvc/llvm-project?rev=361099&view=rev Log: [analyzer] PR41753: Include complex integer types in NonLoc::isCompoundType https://bugs.llvm.org/show_bug.cgi?id=41753 Differential Revision: https://reviews.l

[PATCH] D61570: [analyzer] PR41753: Include complex integer types in NonLoc::isCompoundType

2019-05-18 Thread Kristóf Umann via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL361099: [analyzer] PR41753: Include complex integer types in NonLoc::isCompoundType (authored by Szelethus, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed p

[PATCH] D61816: [CFG] [analyzer] pr41300: Add a branch to skip virtual base initializers when they are handled by the superclass.

2019-05-18 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus requested changes to this revision. Szelethus added a comment. This revision now requires changes to proceed. The more I think about this, the more I think it would be great to see this tested :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61816/new/ https://reviews.llvm.or

[PATCH] D61838: [Sema] Suppress additional warnings for C's zero initializer

2019-05-18 Thread Alex James via Phabricator via cfe-commits
al3xtjames added a comment. Thanks for reviewing! I don't have commit access, so I can't commit this patch myself. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61838/new/ https://reviews.llvm.org/D61838 ___ cfe-com

[PATCH] D61838: [Sema] Suppress additional warnings for C's zero initializer

2019-05-18 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith added inline comments. Comment at: clang/lib/AST/Expr.cpp:2092-2096 const IntegerLiteral *Lit = dyn_cast(getInit(0)); + if (!Lit) { +if (const ImplicitCastExpr *ICE = dyn_cast(getInit(0))) + Lit = dyn_cast(ICE->getSubExpr()); + } Use `Expr::I

[PATCH] D61838: [Sema] Suppress additional warnings for C's zero initializer

2019-05-18 Thread Alex James via Phabricator via cfe-commits
al3xtjames updated this revision to Diff 200159. al3xtjames added a comment. Switched to using Expr::IgnoreImplicit() instead of manually checking for an ImplicitCastExpression. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61838/new/ https://reviews.llvm.org/D6

r361109 - [X86] Remove semicolons at the end of intrinsics implemented as macros so they can be used as arguments to other intrinsics.

2019-05-18 Thread Craig Topper via cfe-commits
Author: ctopper Date: Sat May 18 18:01:52 2019 New Revision: 361109 URL: http://llvm.org/viewvc/llvm-project?rev=361109&view=rev Log: [X86] Remove semicolons at the end of intrinsics implemented as macros so they can be used as arguments to other intrinsics. Also fix one intrinsic that was using

[clang-tools-extra] r361112 - [clangd] Respect clang-tidy suppression comments

2019-05-18 Thread Fangrui Song via cfe-commits
Author: maskray Date: Sat May 18 21:06:52 2019 New Revision: 361112 URL: http://llvm.org/viewvc/llvm-project?rev=361112&view=rev Log: [clangd] Respect clang-tidy suppression comments This partially fixes https://bugs.llvm.org/show_bug.cgi?id=41218. Reviewed By: sammccall Patch by Nathan Ridge!

[PATCH] D60953: [clangd] Respect clang-tidy suppression comments

2019-05-18 Thread Fangrui Song via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL361112: [clangd] Respect clang-tidy suppression comments (authored by MaskRay, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://rev

[clang-tools-extra] r361113 - [clangd] Respect WarningsAsErrors configuration for clang-tidy

2019-05-18 Thread Fangrui Song via cfe-commits
Author: maskray Date: Sat May 18 21:19:14 2019 New Revision: 361113 URL: http://llvm.org/viewvc/llvm-project?rev=361113&view=rev Log: [clangd] Respect WarningsAsErrors configuration for clang-tidy This completes the fix for https://bugs.llvm.org/show_bug.cgi?id=41218. Reviewed By: sammccall Pat

[PATCH] D61841: [clangd] Respect WarningsAsErrors configuration for clang-tidy

2019-05-18 Thread Fangrui Song via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL361113: [clangd] Respect WarningsAsErrors configuration for clang-tidy (authored by MaskRay, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: