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
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
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
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
_
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
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
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
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
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
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
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
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
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!
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
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
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:
16 matches
Mail list logo