Author: djasper
Date: Mon Feb 1 05:20:55 2016
New Revision: 259351
URL: http://llvm.org/viewvc/llvm-project?rev=259351&view=rev
Log:
clang-format: Fix alignment of trailing multiline columns.
Modified:
cfe/trunk/lib/Format/WhitespaceManager.cpp
cfe/trunk/unittests/Format/FormatTest.cpp
Author: djasper
Date: Mon Feb 1 05:21:07 2016
New Revision: 259353
URL: http://llvm.org/viewvc/llvm-project?rev=259353&view=rev
Log:
clang-format: Fix incorrect pointer detection in lambdas in constructor
initializers.
Before:
Constructor() : member([](A *a, B * b) {}) {}
After:
Constructor
Author: djasper
Date: Mon Feb 1 05:20:47 2016
New Revision: 259350
URL: http://llvm.org/viewvc/llvm-project?rev=259350&view=rev
Log:
clang-format: [JS] Treat "in" as a proper operator.
Modified:
cfe/trunk/lib/Format/TokenAnnotator.cpp
cfe/trunk/unittests/Format/FormatTestJS.cpp
Modified
d.zobnin.bugzilla updated this revision to Diff 46521.
d.zobnin.bugzilla added a comment.
Thanks for the review!
Only updated the text of diagnostics, NFC.
http://reviews.llvm.org/D16219
Files:
include/clang/Basic/Attr.td
include/clang/Basic/DiagnosticSemaKinds.td
include/clang/Sema/Attr
nemanjai added a comment.
If the reviewers don't mind, I would like to keep this patch with diagnostics
for interoperability between the two types for now. This is simply because
enabling such interoperability requires changes to some of the conversion
infrastructure (i.e. allowing FPTrunc/FPEx
NewProggie created this revision.
NewProggie added reviewers: poiru, djasper, klimek.
NewProggie added a subscriber: cfe-commits.
Herald added a subscriber: klimek.
This patch adds an option to remove the blank after comma between several
function parameters. The default value is still to keep th
Author: dergachev
Date: Mon Feb 1 03:29:17 2016
New Revision: 259345
URL: http://llvm.org/viewvc/llvm-project?rev=259345&view=rev
Log:
[analyzer] Use a wider integer type for an array index.
Avoids unexpected overflows while performing pointer arithmetics in 64-bit code.
Moreover, neither Pointe
Hi all,
It’s been 10 days since I submitted this patch, without any response; is
there anything else I need to provide/do to get a review and/or commit?
Yours,
Russ Magee %-)
On Thu, Jan 21, 2016 at 12:43 PM, Russell Keith-Magee <
russ...@keith-magee.com> wrote:
> Hi all,
>
> Following some fee
Author: d0k
Date: Mon Feb 1 07:22:39 2016
New Revision: 259355
URL: http://llvm.org/viewvc/llvm-project?rev=259355&view=rev
Log:
Remove the egregious PCHContainer layering hack that doesn't seem to be
necessary anymore.
Modified:
cfe/trunk/lib/Basic/FileManager.cpp
cfe/trunk/lib/Fronten
yaxunl updated this revision to Diff 46526.
yaxunl added a comment.
update the tests.
http://reviews.llvm.org/D16686
Files:
include/clang/Basic/Attr.td
include/clang/Basic/AttrDocs.td
include/clang/Basic/DiagnosticParseKinds.td
include/clang/Basic/DiagnosticSemaKinds.td
include/clang/
Author: aaronballman
Date: Mon Feb 1 08:11:47 2016
New Revision: 259359
URL: http://llvm.org/viewvc/llvm-project?rev=259359&view=rev
Log:
Reapply r259210 with a fix for RegistryTest.cpp.
Patch by Richard Thomson.
Modified:
cfe/trunk/docs/LibASTMatchersReference.html
cfe/trunk/include/cl
aaron.ballman added a comment.
I haven't quite figured out the proper phab workflow for reverts yet either,
but I took the updated patch, applied it, and commit in r259359. Thank you for
the fix!
http://reviews.llvm.org/D8149
___
cfe-commits maili
d.zobnin.bugzilla created this revision.
d.zobnin.bugzilla added reviewers: rnk, majnemer.
d.zobnin.bugzilla added a subscriber: cfe-commits.
Allow typedef redefinition with different types in C if the types are equally
qualified, sized and aligned. MSVC allows such redefinition, emits warning
C
alexfh added a comment.
Nice! See a few comments inline.
Comment at: clang-tidy/misc/IncorrectRoundings.cpp:38
@@ +37,3 @@
+ // Match a floating point expression.
+ auto FloatType = expr(anyOf(hasType(qualType(asString("long double"))),
+ hasType(q
ichesnokov removed rL LLVM as the repository for this revision.
ichesnokov updated this revision to Diff 46532.
ichesnokov added a comment.
Added generic variable support and checking address space to Microsoft mangler.
Test case updated.
http://reviews.llvm.org/D16539
Files:
lib/AST/ItaniumM
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
Aside from one minor nit with `auto` usage, LGTM!
Comment at: C:/LLVM/llvm/tools/clang/lib/StaticAnalyzer/Core/MemRegion.cpp:1396
@@ -1395,3 +1395,3 @@
const
alexfh added a comment.
Thanks! A few comments.
Comment at: clang-tidy/google/NonConstReferences.cpp:54
@@ +53,3 @@
+
+ QualType ReferencedType =
+ *Result.Nodes.getNodeAs("referenced_type");
This could be `auto` to avoid duplicating the type name.
==
aaron.ballman added a comment.
In http://reviews.llvm.org/D16708#340359, @gottesmm wrote:
> I think that my response via email did not hit phabriactor. So sorry for the
> delay.
No worries!
> Yes there is a forthcoming patch for CodeGen which will place an attribute on
> the relevant functio
aaron.ballman added inline comments.
Comment at: clang-tidy/modernize/RawStringLiteralCheck.cpp:80
@@ +79,3 @@
+return false;
+
+ const size_t NewLinePos = Text.find(R"(\n)");
This is why I would still prefer to block on fixing StringLiteral. This is
functio
Author: aaronballman
Date: Mon Feb 1 09:31:15 2016
New Revision: 259362
URL: http://llvm.org/viewvc/llvm-project?rev=259362&view=rev
Log:
Add a new check, readability-redundant-control-flow, that check for some forms
of redundant control flow statements. Currently checks for return statements at
aaron.ballman closed this revision.
aaron.ballman added a comment.
I have commit in r259362. Thank you!
http://reviews.llvm.org/D16259
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
With one small nit to the diagnostic wording for consistency, LGTM! I will go
ahead and make that change, then commit on your behalf. You may want to talk to
Chris Lattner about
aaron.ballman added inline comments.
Comment at: clang-tidy/readability/SimplifyBooleanExprCheck.h:21-22
@@ -20,71 +20,4 @@
/// them to use the appropriate boolean expression directly.
///
-/// Examples:
-///
-/// ===
-//
hokein updated this revision to Diff 46539.
hokein added a comment.
Address Alex's comments.
http://reviews.llvm.org/D16764
Files:
clang-tidy/misc/CMakeLists.txt
clang-tidy/misc/IncorrectRoundings.cpp
clang-tidy/misc/IncorrectRoundings.h
clang-tidy/misc/MiscTidyModule.cpp
docs/clang-t
ayartsev updated this revision to Diff 46538.
ayartsev added a comment.
Updated the patch, please review.
http://reviews.llvm.org/D16317
Files:
lib/StaticAnalyzer/Checkers/CStringChecker.cpp
test/Analysis/string.c
Index: test/Analysis/string.c
==
hokein marked 3 inline comments as done.
Comment at: clang-tidy/misc/IncorrectRoundings.cpp:39
@@ +38,3 @@
+namespace tidy {
+void IncorrectRoundings::registerMatchers(MatchFinder *MatchFinder) {
+ // Match a floating literal with value 0.5.
Done. The `ASTMatcher
>> Yes, that is what I mean. It is odd that -frtti changes us from "this
>> is not available anywhere, use linkonce_odr" to "it is available
>> elsewhere, use an external declaration".
>
> Yes, I agree, it's weird (although the transition is in the other
> direction, really, since there's no such f
alexfh added inline comments.
Comment at: clang-tidy/modernize/RawStringLiteralCheck.cpp:80
@@ +79,3 @@
+return false;
+
+ const size_t NewLinePos = Text.find(R"(\n)");
aaron.ballman wrote:
> This is why I would still prefer to block on fixing StringLiteral.
Author: arpith
Date: Mon Feb 1 10:32:47 2016
New Revision: 259366
URL: http://llvm.org/viewvc/llvm-project?rev=259366&view=rev
Log:
[OpenMP] Prevent nesting of target constructs within target code execution
regions.
Summary:
This patch enhances Sema to check for the following restriction:
Open
majnemer added a comment.
MSVC permits:
typedef int foo;
typedef float foo;
This is crazy and most certainly not 'benign'. I think that we should insist
that the underlying types not just be the same size and alignment but the same
kind of scalar type. If two types are integral or enumerat
This revision was automatically updated to reflect the committed changes.
Closed by commit rL259366: [OpenMP] Prevent nesting of target constructs within
target code execution… (authored by arpith).
Changed prior to commit:
http://reviews.llvm.org/D16758?vs=46496&id=46544#toc
Repository:
rL
alexfh added a comment.
Thank you for adding more test cases! Could you address one more comment in a
follow-up, please?
Comment at: test/clang-tidy/readability-redundant-control-flow.cpp:193
@@ +192,3 @@
+// CHECK-FIXES-NEXT: {{^return;$}}
+// CHECK-FIXES-NEXT: {{^ *}$}}
+
Anastasia accepted this revision.
Anastasia added a comment.
This revision is now accepted and ready to land.
LGTM!
http://reviews.llvm.org/D16692
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
rmaprath added a comment.
Ping?
http://reviews.llvm.org/D16586
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
LegalizeAdulthood added inline comments.
Comment at: clang-tidy/modernize/RawStringLiteralCheck.cpp:88
@@ +87,3 @@
+}
+
+bool containsDelimiter(StringRef Bytes, const std::string &Delimiter) {
alexfh wrote:
> aaron.ballman wrote:
> > I think Alex's point is: why n
LegalizeAdulthood added inline comments.
Comment at: clang-tidy/readability/SimplifyBooleanExprCheck.h:21-22
@@ -20,71 +20,4 @@
/// them to use the appropriate boolean expression directly.
///
-/// Examples:
-///
-/// ===
aaron.ballman added inline comments.
Comment at: clang-tidy/readability/SimplifyBooleanExprCheck.h:21-22
@@ -20,71 +20,4 @@
/// them to use the appropriate boolean expression directly.
///
-/// Examples:
-///
-/// ===
-//
rnk added inline comments.
Comment at: include/clang/AST/ASTContext.h:1771-1782
@@ -1770,1 +1770,14 @@
+ /// Return true is the given types are compatible in C from MSVC's point of
+ /// view.
+ //
+ // Conditions:
+ // 1. Both types are equally-qualified, sized and align
On Fri, Jan 29, 2016 at 4:50 PM, Alexey Samsonov wrote:
>
> On Fri, Jan 29, 2016 at 3:16 PM, Hans Wennborg wrote:
>>
>> Yes, that seems like a good idea. Go ahead and merge (or let me know
>> if you'd prefer me to do it).
>
>
> I'd appreciate if you could do it. Thanks!
r259371. Thanks.
>> On F
LegalizeAdulthood added a comment.
In http://reviews.llvm.org/D16526#334938, @aaron.ballman wrote:
> I'm of two minds on this [...]
I just need a thumbs up/down on this.
Thumbs down, I discard the review.
Thumbs up, you take the patch.
Patch by Richard Thomson.
http://reviews.llvm.org/D165
aaron.ballman added a comment.
In http://reviews.llvm.org/D16526#340785, @LegalizeAdulthood wrote:
> In http://reviews.llvm.org/D16526#334938, @aaron.ballman wrote:
>
> > I'm of two minds on this [...]
>
>
> I just need a thumbs up/down on this.
>
> Thumbs down, I discard the review.
>
> Thumbs u
aprantl added inline comments.
Comment at: lib/CodeGen/CGDebugInfo.cpp:2099
@@ -2098,1 +2098,3 @@
+llvm::DIMacro *CGDebugInfo::CreateMacro(llvm::DIMacroFile *Parent, bool
IsUndef,
+SourceLocation LineLoc, StringRef Name,
Author: d0k
Date: Mon Feb 1 11:42:01 2016
New Revision: 259376
URL: http://llvm.org/viewvc/llvm-project?rev=259376&view=rev
Log:
Move LocInfoType from Sema to AST.
While transient and only used during parsing, LocInfoTypes are still used
from ASTDumper and are part of the AST.
Added:
cfe/tr
jpienaar added a comment.
@jlebar: We defer it to your and Richard's approval. Thanks
http://reviews.llvm.org/D15305
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Anastasia added inline comments.
Comment at: include/clang/Parse/Parser.h:2200
@@ -2199,1 +2199,3 @@
void ParseOpenCLQualifiers(ParsedAttributes &Attrs);
+ /// \brief Parses opencl_unroll_hint attribute if language is OpenCL 2.0+.
+ /// \return false if error happens.
---
hjl.tools created this revision.
hjl.tools added reviewers: rnk, rafael.
hjl.tools added a subscriber: cfe-commits.
```
typedef unsigned int gcc_word __attribute__((mode(word)));
```
and
```
typedef unsigned int gcc_unwind_word __attribute__((mode(unwind_word)));
```
define the largest unsigned i
Anastasia added a subscriber: Anastasia.
Comment at: lib/Sema/SemaDecl.cpp:2038
@@ +2037,3 @@
+ // returns 0. The're many implicit typedefs in OpenCL, e.g. atomic_flag.
+ if (Old->isImplicit() || New->isImplicit()) {
+return;
Braces are not needed.
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
Should I land this for you?
http://reviews.llvm.org/D16779
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
hjl.tools added a comment.
Yes, please. Thanks.
http://reviews.llvm.org/D16779
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: pcc
Date: Mon Feb 1 12:55:50 2016
New Revision: 259382
URL: http://llvm.org/viewvc/llvm-project?rev=259382&view=rev
Log:
docs: Clarify that cfi-unrelated-cast is based on lifetime.
Also restore Makefile.sphinx which is needed to build the documentation.
Added:
cfe/trunk/docs/Makefil
Author: rnk
Date: Mon Feb 1 12:58:24 2016
New Revision: 259383
URL: http://llvm.org/viewvc/llvm-project?rev=259383&view=rev
Log:
Fix attribute((mode([word|unwind_word]))) for x32
Patch by H.J. Lu
```
typedef unsigned int gcc_word __attribute__((mode(word)));
```
and
```
typedef unsigned int gc
This revision was automatically updated to reflect the committed changes.
Closed by commit rL259383: Fix attribute((mode([word|unwind_word]))) for x32
(authored by rnk).
Changed prior to commit:
http://reviews.llvm.org/D16779?vs=46558&id=46559#toc
Repository:
rL LLVM
http://reviews.llvm.org
Author: eugenezelenko
Date: Mon Feb 1 13:47:24 2016
New Revision: 259393
URL: http://llvm.org/viewvc/llvm-project?rev=259393&view=rev
Log:
Sort checks alphabetically in ReadabilityTidyModule.cpp.
Modified:
clang-tools-extra/trunk/clang-tidy/readability/ReadabilityTidyModule.cpp
Modified:
c
> Hi Anton,
>
> Okey, nevermore.
> But what to do with a bug which needs simply be closed (NABs)?
> I believe they also need a review/approval.
>
> Igor
Bugs do not have such a strict process, compared to patches.
If you don't feel confident in closing the bug directly, you can
yaxunl updated this revision to Diff 46570.
yaxunl marked 7 inline comments as done.
yaxunl added a comment.
Revised as Anastasia suggested.
http://reviews.llvm.org/D16686
Files:
include/clang/Basic/Attr.td
include/clang/Basic/AttrDocs.td
include/clang/Basic/DiagnosticParseKinds.td
incl
Thanks Paul!
-Original Message-
From: Robinson, Paul [mailto:paul_robin...@playstation.sony.com]
Sent: Monday, February 1, 2016 11:46 PM
To: Igor Chesnokov; Anton Korobeynikov
Cc: cfe-commits (cfe-commits@lists.llvm.org)
Subject: RE: [PATCH] D16754: Bug 15785 - OpenCL errors using vector/
Richard,
On Fri, Jan 15, 2016 at 5:32 PM, Richard Smith
wrote:
> On Fri, Jan 15, 2016 at 5:29 PM, Richard Smith
> wrote:
> > On Fri, Jan 15, 2016 at 4:22 PM, Artem Belevich wrote:
> >> tra added inline comments.
> >>
> >>
> >> Comment at: lib/CodeGen/CodeGenModule.cpp:2334
> >
rsmith added a comment.
This is a bug -- we intend to reuse `Stmt` nodes across the template and its
instantiations, but we should have separate `Decl` nodes in each instantiation
(otherwise the `Decl`'s parent will be wrong etc).
Comment at: lib/Sema/SemaTemplateInstantiate.c
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Some minor things, but feel free to commit after addressing them.
I agree that we should figure out what to do about the zero/undef
initialization separately.
Comment at: li
ariccio marked an inline comment as done.
ariccio added a comment.
As said in comment, I disagree with the no need for `const` here.
Comment at: C:/LLVM/llvm/tools/clang/lib/StaticAnalyzer/Core/MemRegion.cpp:1396
@@ -1395,3 +1395,3 @@
const auto &ReferencedBlockVars = AC->get
eugenis added a comment.
In http://reviews.llvm.org/D16738#340175, @EricWF wrote:
> > This also could be fixed in a different way by replacing C-style
>
> > casts with reinterpret_cast<>, which, from my reading of the
>
> > standard, is allowed in this context.
>
>
> I agree that using `void*`
Eugene.Zelenko created this revision.
Eugene.Zelenko added reviewers: alexfh, LegalizeAdulthood, aaron.ballman.
Eugene.Zelenko added a subscriber: cfe-commits.
Eugene.Zelenko set the repository for this revision to rL LLVM.
This fix for PR25894. I checked it on my work code base.
Build and regres
aaron.ballman added inline comments.
Comment at: C:/LLVM/llvm/tools/clang/lib/StaticAnalyzer/Core/MemRegion.cpp:1396
@@ -1395,3 +1395,3 @@
const auto &ReferencedBlockVars = AC->getReferencedBlockVars(BC->getDecl());
- auto NumBlockVars =
+ const auto NumBlockVars =
std
sepavloff updated this revision to Diff 46575.
sepavloff added a comment.
Added regression test.
http://reviews.llvm.org/D15450
Files:
lib/CodeGen/CodeGenAction.cpp
unittests/Frontend/CMakeLists.txt
unittests/Frontend/CodeGenActionTest.cpp
Index: unittests/Frontend/CodeGenActionTest.cpp
LGTM for 3.8.
On Wed, Jan 13, 2016 at 4:03 PM, Hans Wennborg wrote:
> Thanks! Richard, what say the code owner?
>
> On Wed, Jan 13, 2016 at 3:59 PM, George Burgess IV
> wrote:
>> FYI: This patch should to be merged into the clang 3.8.0 branch.
>>
>> Thank you!
>> George
>>
>> On Wed, Jan 13, 201
Author: aaronballman
Date: Mon Feb 1 15:28:33 2016
New Revision: 259409
URL: http://llvm.org/viewvc/llvm-project?rev=259409&view=rev
Log:
Code clean up; NFC.
Patch by Alexander Riccio.
Modified:
cfe/trunk/lib/StaticAnalyzer/Core/CheckerRegistry.cpp
Modified: cfe/trunk/lib/StaticAnalyzer/Co
LGTM for 3.8.
On Thu, Jan 21, 2016 at 1:10 PM, Hans Wennborg wrote:
> Richard, it was suggested (in
> https://llvm.org/bugs/show_bug.cgi?id=26059#c7) that this gets merged
> to 3.8. I believe this falls under your ownership.
>
> On Thu, Jan 21, 2016 at 4:54 AM, Alexey Bataev via cfe-commits
> wr
LegalizeAdulthood added inline comments.
Comment at: clang-tidy/readability/SimplifyBooleanExprCheck.h:21-22
@@ -20,71 +20,4 @@
/// them to use the appropriate boolean expression directly.
///
-/// Examples:
-///
-/// ===
C++11 narrowing only happens during certain conversions while this warning
checks all conversions.
We might be able to avoid char array initialization, but it would be a
little tricky. These warning usually have little information about where
the conversion is coming from, so distinguishing array
Thanks! r259412.
On Mon, Feb 1, 2016 at 1:32 PM, Richard Smith wrote:
> LGTM for 3.8.
>
> On Wed, Jan 13, 2016 at 4:03 PM, Hans Wennborg wrote:
>> Thanks! Richard, what say the code owner?
>>
>> On Wed, Jan 13, 2016 at 3:59 PM, George Burgess IV
>> wrote:
>>> FYI: This patch should to be merged
aaron.ballman added inline comments.
Comment at: clang-tidy/readability/SimplifyBooleanExprCheck.h:21-22
@@ -20,71 +20,4 @@
/// them to use the appropriate boolean expression directly.
///
-/// Examples:
-///
-/// ===
-//
Thanks! r259414.
On Mon, Feb 1, 2016 at 1:36 PM, Richard Smith wrote:
> LGTM for 3.8.
>
> On Thu, Jan 21, 2016 at 1:10 PM, Hans Wennborg wrote:
>> Richard, it was suggested (in
>> https://llvm.org/bugs/show_bug.cgi?id=26059#c7) that this gets merged
>> to 3.8. I believe this falls under your own
Marshall: ping?
On Tue, Jan 26, 2016 at 11:08 AM, Hans Wennborg wrote:
> On Tue, Jan 19, 2016 at 9:21 AM, Hans Wennborg wrote:
>> On Tue, Jan 19, 2016 at 12:01 AM, Dimitry Andric wrote:
>>> On 19 Jan 2016, at 01:50, Marshall Clow via cfe-commits
>>> wrote:
Author: marshall
Date
On Mon, Jan 25, 2016 at 8:03 PM, Xiuli Pan via cfe-commits
wrote:
> Author: pxl
> Date: Mon Jan 25 22:03:48 2016
> New Revision: 258782
>
> URL: http://llvm.org/viewvc/llvm-project?rev=258782&view=rev
> Log:
> Recommit: R258773 [OpenCL] Pipe builtin functions
> Fix arc patch fuzz error.
> Summary:
On Thu, Jan 28, 2016 at 11:25 AM, Hans Wennborg wrote:
> I don't think we have a specific code owner for OpenCL in Clang, which
> means Richard is the owner.
>
> Richard, what do you think?
Is there a reason we want to push this new feature into 3.8 rather
than waiting for the next release?
> On
thakis updated this revision to Diff 46577.
thakis added a comment.
just return true
http://reviews.llvm.org/D16478
Files:
lib/Sema/SemaTemplateInstantiate.cpp
Index: lib/Sema/SemaTemplateInstantiate.cpp
===
--- lib/Sema/SemaTem
thakis marked an inline comment as done.
Comment at: lib/Sema/SemaTemplateInstantiate.cpp:1519-1521
@@ -1518,5 +1518,5 @@
TypeLoc TL = T->getTypeLoc().IgnoreParens();
if (!TL.getAs())
return false;
Ack.
http://reviews.llvm.org/D16478
__
On Mon, Feb 1, 2016 at 1:53 PM, Richard Smith wrote:
> On Thu, Jan 28, 2016 at 11:25 AM, Hans Wennborg wrote:
>> I don't think we have a specific code owner for OpenCL in Clang, which
>> means Richard is the owner.
>>
>> Richard, what do you think?
>
> Is there a reason we want to push this new f
thakis updated this revision to Diff 46578.
thakis added a comment.
restore accidentally dropped test
http://reviews.llvm.org/D16478
Files:
lib/Sema/SemaTemplateInstantiate.cpp
unittests/ASTMatchers/ASTMatchersTest.cpp
Index: unittests/ASTMatchers/ASTMatchersTest.cpp
==
Author: arpith
Date: Mon Feb 1 16:02:05 2016
New Revision: 259418
URL: http://llvm.org/viewvc/llvm-project?rev=259418&view=rev
Log:
Undoing commit r259366 to debug buildbot failure.
> http://reviews.llvm.org/D16758
Modified:
cfe/trunk/include/clang/Basic/OpenMPKinds.h
cfe/trunk/lib/Basi
On Thu, Jan 14, 2016 at 2:57 PM, Richard Trieu via cfe-commits
wrote:
> Author: rtrieu
> Date: Thu Jan 14 16:56:39 2016
> New Revision: 257831
>
> URL: http://llvm.org/viewvc/llvm-project?rev=257831&view=rev
> Log:
> Refactor template type diffing
>
> 1) Instead of using pairs of From/To* fields,
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM
Comment at: lib/Sema/SemaTemplateInstantiate.cpp:1525-1530
@@ -1524,18 +1524,8 @@
for (unsigned I = 0, E = FP.getNumParams(); I != E; ++I) {
-ParmVarDecl *P = FP.g
Author: matze
Date: Mon Feb 1 16:18:58 2016
New Revision: 259424
URL: http://llvm.org/viewvc/llvm-project?rev=259424&view=rev
Log:
Fix build problem by lower SmallSet to a reasonable value
Modified:
clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp
Modified: clang-tools-extra/trunk
Author: nico
Date: Mon Feb 1 16:31:51 2016
New Revision: 259428
URL: http://llvm.org/viewvc/llvm-project?rev=259428&view=rev
Log:
Always build a new TypeSourceInfo for function templates with parameters
RecursiveASTVisitor::TraverseFunctionHelper() traverses a function's
ParmVarDecls by going to
thakis closed this revision.
thakis added a comment.
r259428, thanks! Went with the for-each loop but left the structure as-is -- I
find the comments helpful and it's not clear where they should go else.
http://reviews.llvm.org/D16478
___
cfe-commi
Hello everyone,
LLVM buildmaster will be updated and restarted after 7 PM Pacific time
today.
Thanks
Galina
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
LegalizeAdulthood added inline comments.
Comment at: clang-tidy/readability/SimplifyBooleanExprCheck.h:21-22
@@ -20,71 +20,4 @@
/// them to use the appropriate boolean expression directly.
///
-/// Examples:
-///
-/// ===
nwilson updated this revision to Diff 46588.
nwilson added a comment.
- Fix a couple of comments to reflect the Patch.
- Clang-format the changes in this Patch.
http://reviews.llvm.org/D13357
Files:
include/clang/AST/DeclTemplate.h
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaD
Sunil_Srivastava created this revision.
Sunil_Srivastava added reviewers: rjmccall, DmitryPolukhin, rsmith, probinson.
Sunil_Srivastava added a subscriber: cfe-commits.
This is the round 2 of the PS4 ABI. In this round:
1) A new value PS4 has been added to TargetCXXABI::Kind. It is being used for
nwilson marked 5 inline comments as done.
nwilson added a comment.
Marking some comments Done which were fixed in previous updates.
http://reviews.llvm.org/D13357
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/
Ping.
Marshall, does this look okay?
> On 2016-Jan-25, at 10:32, Duncan P. N. Exon Smith via cfe-commits
> wrote:
>
> dexonsmith updated this revision to Diff 45879.
> dexonsmith added a comment.
>
> Addressed Marshall's review comments: deferring to
> match_results::operator[]() rather than
dexonsmith added a subscriber: dexonsmith.
dexonsmith added a comment.
Ping.
Marshall, does this look okay?
http://reviews.llvm.org/D16467
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
Author: rtrieu
Date: Mon Feb 1 18:36:59 2016
New Revision: 259445
URL: http://llvm.org/viewvc/llvm-project?rev=259445&view=rev
Log:
Template Type Diffing change
When all the arguments of a template are elided, print "A<...>" instead of
"A<[2 * ...]>". Also remove comment fragment that means not
On Mon, Feb 1, 2016 at 2:10 PM, Richard Smith wrote:
> On Thu, Jan 14, 2016 at 2:57 PM, Richard Trieu via cfe-commits
> wrote:
> > Author: rtrieu
> > Date: Thu Jan 14 16:56:39 2016
> > New Revision: 257831
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=257831&view=rev
> > Log:
> > Refactor
rsmith added a subscriber: rsmith.
Comment at: test/CodeGen/aapcs-bitfield.c:312-317
@@ +311,8 @@
+
+ // BE: %[[PTR3:.*]] = bitcast %struct.st5a* %[[PTR2]] to i32*
+ // BE-NEXT: %[[LD:.*]] = load volatile i32, i32* %[[PTR3]], align 4
+ // BE-NEXT: %[[CLR:.*]] = and i32 %[[LD]],
rsmith added inline comments.
Comment at: lib/AST/RecordLayoutBuilder.cpp:1598-1599
@@ -1597,1 +1597,4 @@
+// PS4 remains compatible to pre r257462 behavior.
+bool isPS4ABI = (Context.getTargetInfo().getCXXABI().getKind() ==
TargetCXXABI::PS4);
+
Please
dexonsmith created this revision.
dexonsmith added a reviewer: EricWF.
dexonsmith added a subscriber: cfe-commits.
When `!defined(_LIBCPP_DEBUG)`, unordered_map::emplace_hint() forwards
to emplace(). Do the same when `defined(_LIBCPP_DEBUG)`.
This has no real functionality change, just unifies t
dexonsmith created this revision.
dexonsmith added a reviewer: EricWF.
dexonsmith added a subscriber: cfe-commits.
Instead of duplicating code in unordered_map::emplace(), use
__hash_table::__emplace_unique().
http://reviews.llvm.org/D16792
Files:
include/unordered_map
Index: include/unordere
compnerd added a comment.
I think it would be better if we could actually create a helper to get the
profiling library name for a specific library, and use that in both locations.
Do we have a test for the math case at the very least?
http://reviews.llvm.org/D16264
_
On Thu, Jan 28, 2016 at 8:23 AM, Vassil Vassilev wrote:
> Would this patch be more reasonable? It follows what
> RegisterTemplateSpecialization (introduced in r245779) does. AFAICT this
> adds an update record far less often.
It's still adding redundant update records. We'll write the
appropriate
1 - 100 of 127 matches
Mail list logo
