ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
https://reviews.llvm.org/D27488
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
https://reviews.llvm.org/D27345
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
baloghadamsoftware updated this revision to Diff 80728.
baloghadamsoftware marked 4 inline comments as done.
baloghadamsoftware added a comment.
Minor corrections, comments, some new tests, test input headers merged.
https://reviews.llvm.org/D25660
Files:
include/clang/StaticAnalyzer/Checkers
baloghadamsoftware added a comment.
https://reviews.llvm.org/D27202 is now a dependency, but cannot add it.
Comment at: lib/StaticAnalyzer/Checkers/IteratorPastEndChecker.cpp:166
+IteratorPastEndChecker::IteratorPastEndChecker() {
+ PastEndBugType.reset(new BugType(this, "Iter
Author: ericwf
Date: Thu Dec 8 03:57:00 2016
New Revision: 289033
URL: http://llvm.org/viewvc/llvm-project?rev=289033&view=rev
Log:
Avoid C++17 guaranteed copy elision when testing for non-copyability
Modified:
libcxx/trunk/test/std/input.output/stream.buffers/streambuf/streambuf.cons/copy.
baloghadamsoftware added a comment.
Could you please remove the IteratorPastEndChecker file differences from this
patch and make https://reviews.llvm.org/D25660 dependent on this one?
https://reviews.llvm.org/D27202
___
cfe-commits mailing list
cfe
Alpha updated this revision to Diff 80732.
Alpha added a comment.
Rebase on top of HEAD.
Ping.
Repository:
rL LLVM
https://reviews.llvm.org/D26137
Files:
include/clang/Tooling/Core/Diagnostic.h
include/clang/Tooling/Core/Replacement.h
include/clang/Tooling/DiagnosticsYaml.h
include/c
Author: ericwf
Date: Thu Dec 8 04:02:04 2016
New Revision: 289034
URL: http://llvm.org/viewvc/llvm-project?rev=289034&view=rev
Log:
Add more test cases to packaged_task copyability test
Modified:
libcxx/trunk/test/std/thread/futures/futures.task/futures.task.members/ctor1.fail.cpp
Modified
echuraev created this revision.
echuraev added a reviewer: Anastasia.
echuraev added subscribers: bader, yaxunl, cfe-commits.
Enabling the compression of CLK_NULL_QUEUE to variable of type queue_t.
https://reviews.llvm.org/D27569
Files:
lib/Sema/SemaExpr.cpp
test/CodeGenOpenCL/null_queue.cl
klimek added inline comments.
Comment at: tools/clang-format-vs/ClangFormat/ClangFormat.vsct:76
+
+ Clang Format Document
+
hans wrote:
> amaiorano wrote:
> > hans wrote:
> > > I think File would be better than Document when referring to
malcolm.parsons updated this revision to Diff 80734.
malcolm.parsons added a comment.
Herald added a subscriber: JDevlieghere.
Handle unary operators.
https://reviews.llvm.org/D26750
Files:
clang-tidy/modernize/CMakeLists.txt
clang-tidy/modernize/ModernizeTidyModule.cpp
clang-tidy/moderni
erikjv updated this revision to Diff 80735.
https://reviews.llvm.org/D21075
Files:
lib/Parse/ParseDecl.cpp
lib/Sema/SemaExpr.cpp
test/Index/diagnostic-ranges.cpp
Index: test/Index/diagnostic-ranges.cpp
===
--- /dev/null
+++ t
klimek accepted this revision.
klimek added a comment.
This revision is now accepted and ready to land.
Looks good. Thanks!
https://reviews.llvm.org/D27447
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman
Author: malcolm.parsons
Date: Thu Dec 8 05:46:22 2016
New Revision: 289042
URL: http://llvm.org/viewvc/llvm-project?rev=289042&view=rev
Log:
[ASTMatcher] Add hasReplacementType matcher for SubstTemplateTypeParmType
Summary: Needed for https://reviews.llvm.org/D27166
Reviewers: sbenza, bkramer,
This revision was automatically updated to reflect the committed changes.
Closed by commit rL289042: [ASTMatcher] Add hasReplacementType matcher for
SubstTemplateTypeParmType (authored by malcolm.parsons).
Changed prior to commit:
https://reviews.llvm.org/D27447?vs=80465&id=80741#toc
Repositor
Author: dergachev
Date: Thu Dec 8 08:05:48 2016
New Revision: 289047
URL: http://llvm.org/viewvc/llvm-project?rev=289047&view=rev
Log:
[analyzer] Add dispatch_data_create as a special case in RetainCountChecker.
This function receives a callback block. The analyzer suspects that this block
may b
This revision was automatically updated to reflect the committed changes.
Closed by commit rL289047: [analyzer] Add dispatch_data_create as a special
case in RetainCountChecker. (authored by dergachev).
Changed prior to commit:
https://reviews.llvm.org/D27409?vs=80606&id=80750#toc
Repository:
NoQ updated this revision to Diff 80751.
NoQ marked an inline comment as done.
NoQ added a comment.
- Address comments
- Richer warning message (is it in good shape?)
- Add tests for lack of implementation
https://reviews.llvm.org/D27535
Files:
include/clang/StaticAnalyzer/Checkers/Checkers.t
NoQ added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/ObjCPropertyChecker.cpp:68
+return;
+
+ BR.EmitBasicReport(
dcoughlin wrote:
> You'll also want to make sure to not warn on the following idiom, in which
> programmers use @synthesize to gen
NoQ updated this revision to Diff 80756.
NoQ added a comment.
Don't construct a `StringRef` to a temporary `std::string`.
https://reviews.llvm.org/D27535
Files:
include/clang/StaticAnalyzer/Checkers/Checkers.td
lib/StaticAnalyzer/Checkers/CMakeLists.txt
lib/StaticAnalyzer/Checkers/ObjCPro
If this fixes the problem on python 3 (and it leaves the current behavior
on python 2), then it looks good. Can you submit it?
On Wed, Dec 7, 2016 at 6:59 AM, Vedant Kumar wrote:
> Hi Alexander,
>
> I think this will cause an error if used with python3 (at least, it fails
> on my
> system). Here
Author: arphaman
Date: Thu Dec 8 08:46:05 2016
New Revision: 289055
URL: http://llvm.org/viewvc/llvm-project?rev=289055&view=rev
Log:
[Sema] Avoid "case value not in enumerated type" warning for C++11 opaque enums
This commit ensures that the switch warning "case value not in enumerated type"
is
This revision was automatically updated to reflect the committed changes.
Closed by commit rL289055: [Sema] Avoid "case value not in enumerated type"
warning for C++11 opaque enums (authored by arphaman).
Changed prior to commit:
https://reviews.llvm.org/D27299?vs=79892&id=80757#toc
Repository
jroelofs added a subscriber: rengolin.
jroelofs added a comment.
In https://reviews.llvm.org/D27123#616887, @saaadhu wrote:
> Make defines for CHAR16_TYPE, {U,}INT_{LEAST,FAST}16_TYPE use int instead of
> short.
>
> {U,}INT16_TYPE still gets defined as short though -
> lib/Frontend/InitPreproce
alexfh added a comment.
The patch spans two repos, so I couldn't apply it using arcanist. Just made it
"manually" (patch -p0 -i ...); now running tests...
Repository:
rL LLVM
https://reviews.llvm.org/D26137
___
cfe-commits mailing list
cfe-commi
Author: arphaman
Date: Thu Dec 8 09:09:40 2016
New Revision: 289058
URL: http://llvm.org/viewvc/llvm-project?rev=289058&view=rev
Log:
[CodeCompletion] Provide Objective-C class property completion results
This commit provides class property code completion results. It supports
explicit and impli
This revision was automatically updated to reflect the committed changes.
arphaman marked an inline comment as done.
Closed by commit rL289058: [CodeCompletion] Provide Objective-C class property
completion results (authored by arphaman).
Changed prior to commit:
https://reviews.llvm.org/D27053
arphaman added a comment.
Ping
Repository:
rL LLVM
https://reviews.llvm.org/D27165
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alexfh requested changes to this revision.
alexfh added a comment.
This revision now requires changes to proceed.
Looks like compiler has found a couple of bugs:
In file included from
llvm.git/tools/clang/tools/extra/clang-tidy/ClangTidy.cpp:38:
llvm.git/tools/clang/include/clang/Tooling/Dia
Author: arphaman
Date: Thu Dec 8 10:49:05 2016
New Revision: 289068
URL: http://llvm.org/viewvc/llvm-project?rev=289068&view=rev
Log:
[CodeCompletion][ObjC] Use a parameterized NSSet return type for
keyPathsForValuesAffecting* KVO completion results
rdar://23791701
Modified:
cfe/trunk/lib/S
rmaprath created this revision.
rmaprath added reviewers: EricWF, mclow.lists.
rmaprath added a subscriber: cfe-commits.
Herald added a subscriber: mgorny.
This is a replacement for https://reviews.llvm.org/D27204. Re-worked so that we
use the threading API of `libcxx` instead of creating a whole
rmaprath abandoned this revision.
rmaprath added a comment.
Abandoning as this has been replaced with https://reviews.llvm.org/D27575.
https://reviews.llvm.org/D27204
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-
rmaprath created this revision.
rmaprath added reviewers: EricWF, mclow.lists.
rmaprath added a subscriber: cfe-commits.
Herald added a subscriber: mgorny.
This patch adds support for https://reviews.llvm.org/D27575. A couple of new
threading API calls has been introduced (libc++abi requires thes
zaks.anna added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/IteratorPastEndChecker.cpp:721
+
+static bool inTopLevelNamespace(const Decl *D, IdentifierInfo *II) {
+ const auto *ND = dyn_cast(D->getDeclContext());
zaks.anna wrote:
> Would isInStdName
rmaprath abandoned this revision.
rmaprath added a comment.
Abandoning (replaced with
https://reviews.llvm.org/D27575/https://reviews.llvm.org/D27576).
https://reviews.llvm.org/D27206
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://l
Author: dyung
Date: Thu Dec 8 11:27:20 2016
New Revision: 289075
URL: http://llvm.org/viewvc/llvm-project?rev=289075&view=rev
Log:
Fixing test to work when the compiler defaults to a different C++ standard
version.
Differential Revision: https://reviews.llvm.org/D27488
Modified:
cfe/trunk
This revision was automatically updated to reflect the committed changes.
Closed by commit rL289075: Fixing test to work when the compiler defaults to a
different C++ standard… (authored by dyung).
Changed prior to commit:
https://reviews.llvm.org/D27488?vs=80496&id=80776#toc
Repository:
rL
Author: kromanova
Date: Thu Dec 8 11:57:23 2016
New Revision: 289083
URL: http://llvm.org/viewvc/llvm-project?rev=289083&view=rev
Log:
[DOXYGEN] Improved doxygen comments.
Improved doxygen comments for __wmmintrin_pclmul.h and ammintrin.h intrinsics
by taagging parameter names with \a doxygen c
jlebar accepted this revision.
jlebar added inline comments.
This revision is now accepted and ready to land.
Comment at: lib/Sema/SemaDecl.cpp:8415
+// specialization inherits its target attributes from its template
+// in CheckFunctionTemplateSpecialization() call below
Eugene.Zelenko added a comment.
How does check handle bit fields? I encountered problem on next code in
ASTContext.h:
struct TypeInfo {
uint64_t Width;
unsigned Align;
bool AlignIsRequired : 1;
TypeInfo() : Width(0), Align(0), AlignIsRequired(false) {}
};
Will be also good idea to
graydon created this revision.
graydon added reviewers: doug.gregor, manmanren, rsmith.
graydon added a subscriber: cfe-commits.
The Swift frontend is acquiring the ability to load non-module PCH files
containing
bridging definitions from C/ObjC. As part of this work, it needs to know which
subm
doug.gregor accepted this revision.
doug.gregor added a comment.
This revision is now accepted and ready to land.
LGTM!
https://reviews.llvm.org/D27580
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/lis
tra updated this revision to Diff 80783.
tra marked 3 inline comments as done.
tra added a comment.
Fixed comments.
https://reviews.llvm.org/D25845
Files:
include/clang/Sema/Sema.h
lib/Sema/SemaCUDA.cpp
lib/Sema/SemaDecl.cpp
lib/Sema/SemaTemplate.cpp
test/SemaCUDA/function-template-ov
tra marked an inline comment as done.
tra added inline comments.
Comment at: lib/Sema/SemaTemplate.cpp:7048
// target attributes into account, we perform target match check
// here and reject candidates that have different target.
if (LangOpts.CUDA &&
---
malcolm.parsons added a comment.
In https://reviews.llvm.org/D26750#617316, @Eugene.Zelenko wrote:
> How does check handle bit fields?
There is no warning for bit fields.
I don't think the C++17 bit field initializer proposal has been accepted yet.
> Will be also good idea to add test case for
jlebar added a comment.
lgtm!
https://reviews.llvm.org/D25845
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yaxunl marked 3 inline comments as done.
yaxunl added inline comments.
Comment at: lib/CodeGen/CGExprScalar.cpp:1532
+ return llvm::ConstantInt::get(ConvertType(DestTy),
+ CGF.getContext().getTargetNullPtrValue(E->getType()));
assert(!DestTy->isBooleanType() &&
Author: tra
Date: Thu Dec 8 13:38:13 2016
New Revision: 289091
URL: http://llvm.org/viewvc/llvm-project?rev=289091&view=rev
Log:
[CUDA] Ignore implicit target attributes during function template instantiation.
Some functions and templates are treated as __host__ __device__ even
when they don't h
This revision was automatically updated to reflect the committed changes.
tra marked an inline comment as done.
Closed by commit rL289091: [CUDA] Ignore implicit target attributes during
function template instantiation. (authored by tra).
Changed prior to commit:
https://reviews.llvm.org/D25845
STL_MSFT added a comment.
It was debatable to me whether this was actually a compiler bug, but I suppose
that it can see size()'s definition and should be able to notice that the
elements aren't accessed. I've filed VSO#300037 "Bogus warning C6001 "Using
uninitialized memory" for array::size()"
This revision was automatically updated to reflect the committed changes.
Closed by commit rL289094: [DebugInfo] Add support for __fp16, float, and
double constants. (authored by dgross).
Changed prior to commit:
https://reviews.llvm.org/D27549?vs=80683&id=80806#toc
Repository:
rL LLVM
http
yaxunl updated the summary for this revision.
yaxunl updated this revision to Diff 80804.
yaxunl marked an inline comment as done.
yaxunl added a comment.
Revised by John's comments.
Added source QualType parameter to performAddrSpaceCast.
Changed linkage of tentative definition of global var wit
Author: dgross
Date: Thu Dec 8 14:02:46 2016
New Revision: 289094
URL: http://llvm.org/viewvc/llvm-project?rev=289094&view=rev
Log:
[DebugInfo] Add support for __fp16, float, and double constants.
Summary:
Partial fix for PR26619.
Prior to this change, a DIGlobalVariable corresponding to a stat
STL_MSFT added a comment.
I can't possibly defend C1XX's behavior in your foo() scenario - but the pair
scenarios being fixed here are different. pair's constructor from (X&&,
Y&&) is perfect forwarding, so while the compiler can see that literals are
being passed to (int&&, int&&), when it ins
STL_MSFT added a comment.
I checked, and Clang 3.8.0 behaves identically to C1XX in the pair scenario.
Here's the test case:
#include
template struct Pair {
A a;
B b;
template Pair(X&& x, Y&& y) :
a(std::forward(x)), b(std::forward(y)) { }
};
int main()
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
Huh. Thanks for the analysis. LGTM.
https://reviews.llvm.org/D27540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org
STL_MSFT marked an inline comment as done.
STL_MSFT added inline comments.
Comment at: test/std/re/re.traits/value.pass.cpp:119
}
for (int c = 'g'; c < 0x; ++c)
{
EricWF wrote:
> Can't we just make `c` a `wchar_t` like above?
Yes (b
STL_MSFT updated the summary for this revision.
STL_MSFT updated this revision to Diff 80810.
STL_MSFT marked an inline comment as done.
STL_MSFT added a comment.
Changed test/std/re/re.traits/value.pass.cpp to iterate with wchar_t as
requested.
https://reviews.llvm.org/D27544
Files:
test/s
STL_MSFT added a comment.
Responded to https://reviews.llvm.org/D27540. tuple is affected just like pair.
https://reviews.llvm.org/D27541
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm
STL_MSFT added a comment.
I could mark these assertions as libcxx-specific, but I tend to believe that
they should be removed outright (even if the natural implementation of a list
without debug checks provides this behavior).
https://reviews.llvm.org/D26626
NoQ updated this revision to Diff 80811.
NoQ added a comment.
Don't yell at read-only properties (no -copy is performed).
Single-quote 'copy' in the error messages.
https://reviews.llvm.org/D27535
Files:
include/clang/StaticAnalyzer/Checkers/Checkers.td
lib/StaticAnalyzer/Checkers/CMakeList
dgross created this revision.
dgross added reviewers: cfe-commits, probinson.
https://reviews.llvm.org/D27549 (partial fix for PR26619) emits a constant
value in the debug
metadata for a floating-point static const that does not exceed 64
bits in size. The regression test accompanying that fix a
Author: dgross
Date: Thu Dec 8 15:15:17 2016
New Revision: 289103
URL: http://llvm.org/viewvc/llvm-project?rev=289103&view=rev
Log:
[DebugInfo] Relax test case for long double constants.
Summary:
D27549 (partial fix for PR26619) emits a constant value in the debug
metadata for a floating-point s
This revision was automatically updated to reflect the committed changes.
Closed by commit rL289103: [DebugInfo] Relax test case for long double
constants. (authored by dgross).
Changed prior to commit:
https://reviews.llvm.org/D27589?vs=80815&id=80817#toc
Repository:
rL LLVM
https://review
rjmccall added inline comments.
Comment at: lib/CodeGen/CGExprScalar.cpp:1532
+ return llvm::ConstantInt::get(ConvertType(DestTy),
+ CGF.getContext().getTargetNullPtrValue(E->getType()));
assert(!DestTy->isBooleanType() && "bool should use PointerToBool");
Author: stl_msft
Date: Thu Dec 8 15:37:47 2016
New Revision: 289105
URL: http://llvm.org/viewvc/llvm-project?rev=289105&view=rev
Log:
[libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible
loss of data", part 1/7.
Given `std::basic_streambuf::int_type __c`, `std::basic_str
Author: stl_msft
Date: Thu Dec 8 15:38:01 2016
New Revision: 289106
URL: http://llvm.org/viewvc/llvm-project?rev=289106&view=rev
Log:
[libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible
loss of data", part 2/7.
These tests for some guy's transparent operator functors we
Author: stl_msft
Date: Thu Dec 8 15:38:44 2016
New Revision: 289110
URL: http://llvm.org/viewvc/llvm-project?rev=289110&view=rev
Log:
[libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible
loss of data", part 6/7.
test/std/algorithms/alg.modifying.operations/alg.random.shu
Author: stl_msft
Date: Thu Dec 8 15:38:14 2016
New Revision: 289107
URL: http://llvm.org/viewvc/llvm-project?rev=289107&view=rev
Log:
[libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible
loss of data", part 3/7.
Add static_cast when constructing pair from (Something,
in
Author: stl_msft
Date: Thu Dec 8 15:38:23 2016
New Revision: 289108
URL: http://llvm.org/viewvc/llvm-project?rev=289108&view=rev
Log:
[libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible
loss of data", part 4/7.
Change char to long and remove some char casts. This preser
Author: stl_msft
Date: Thu Dec 8 15:38:57 2016
New Revision: 289111
URL: http://llvm.org/viewvc/llvm-project?rev=289111&view=rev
Log:
[libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible
loss of data", part 7/7.
test/std/input.output/iostream.format/input.streams/istream
Author: stl_msft
Date: Thu Dec 8 15:38:32 2016
New Revision: 289109
URL: http://llvm.org/viewvc/llvm-project?rev=289109&view=rev
Log:
[libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible
loss of data", part 5/7.
Instead of storing double in double and then truncating to
Prazek added a reviewer: staronj.
Prazek added a comment.
gosh, I found it too late. Jakub started to work on the same problem and he
have made initial check for it.
One of the problems he got was multideclarations.
Does it work for cases like?
long a = lexical_cast(z), b = lexical_cast(y);
O
Author: kromanova
Date: Thu Dec 8 16:10:51 2016
New Revision: 289116
URL: http://llvm.org/viewvc/llvm-project?rev=289116&view=rev
Log:
[DOXYGEN] Improved doxygen comments for emmintrin.h intrinsics.
Tagged parameter names with \a doxygen command to display parameters in italics.
Formatted commen
rnk updated this revision to Diff 80827.
rnk marked an inline comment as done.
rnk added a comment.
- Reuse EH specifier storage in C
- Revert dumper change
https://reviews.llvm.org/D27279
Files:
include/clang/AST/Decl.h
include/clang/Sema/DeclSpec.h
include/clang/Sema/Sema.h
lib/AST/AS
rnk added inline comments.
Comment at: include/clang/Sema/DeclSpec.h:1240
+/// in the prototype. These are generally tag types or enumerators.
+unsigned NumDeclsInPrototype : 8;
+
rnk wrote:
> rsmith wrote:
> > It seems plausible that generated code could
malcolm.parsons added a comment.
In https://reviews.llvm.org/D27166#617621, @Prazek wrote:.
> Does it work for cases like?
Yes; `replaceExpr()` checks that the variable has the same unqualified type as
the initializer and the same canonical type as other variables in the
declaration.
https:
Author: aizatsky
Date: Thu Dec 8 16:25:01 2016
New Revision: 289144
URL: http://llvm.org/viewvc/llvm-project?rev=289144&view=rev
Log:
[sanitizers] lsan+sancov doesn't need ubsanrt (multi def error)
Reviewers: eugenis
Differential Revision: https://reviews.llvm.org/D27594
Modified:
cfe/trun
bruno added a comment.
Any thoughts on this @rsmith ?
https://reviews.llvm.org/D26267
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ahatanak added inline comments.
Comment at: include/clang/AST/Decl.h:3250
+ /// This is true if this struct ends with an array marked 'flexible_array'.
+ bool HasFlexibleArrayAttr : 1;
+
rsmith wrote:
> How is this different from `HasFlexibleArrayMember`? Do we
Author: kromanova
Date: Thu Dec 8 17:32:07 2016
New Revision: 289154
URL: http://llvm.org/viewvc/llvm-project?rev=289154&view=rev
Log:
[DOXYGEN] Improved doxygen comments.
Improved doxygen comments for fxsrintrin.h and mmintrin.h intrinsics by
taagging parameter names with \a doxygen command to
Author: ericwf
Date: Thu Dec 8 17:57:08 2016
New Revision: 289158
URL: http://llvm.org/viewvc/llvm-project?rev=289158&view=rev
Log:
Fix PR27374 - Remove the implicit reduced-arity-extension in tuple.
This patch removes libc++'s tuple extension which allowed it to be
constructed from fewer initia
Author: kromanova
Date: Thu Dec 8 17:58:39 2016
New Revision: 289159
URL: http://llvm.org/viewvc/llvm-project?rev=289159&view=rev
Log:
[DOXYGEN] Improved doxygen comments for xmmintrin.h intrinsics.
Tagged parameter names with \a doxygen command to display parameters in italics.
Formatted commen
Author: eugenezelenko
Date: Thu Dec 8 18:11:53 2016
New Revision: 289160
URL: http://llvm.org/viewvc/llvm-project?rev=289160&view=rev
Log:
[AST, ASTMatchers, Basic] Fix some Clang-tidy modernize and Include What You
Use warnings; other minor fixes (NFC).
Modified:
cfe/trunk/include/clang/AS
Author: probinson
Date: Thu Dec 8 19:20:40 2016
New Revision: 289167
URL: http://llvm.org/viewvc/llvm-project?rev=289167&view=rev
Log:
Specify -std=gnu++98 on some Rewriter tests. NFC.
Rewriter tests rewrite Objective-C++ to C++, and then compile the
result. The rewritten result sometimes doesn
Hi Richard,
at this point this is more a heads-up than anything actionable, but I wanted to
let you know that I bisected this bot failure
(http://lab.llvm.org:8080/green/job/clang-stage2-cmake-RgSan_check/2721/consoleFull#10584592348254eaf0-7326-4999-85b0-388101f2d404)
of std/input.output/strea
(I replied to comments about 1h ago, looks like phab is not in the mood for
sending a mail for that for some reason. I did use the "Leap into action"
submit button.)
On Wed, Dec 7, 2016 at 9:19 PM, Richard Smith via Phabricator <
revi...@reviews.llvm.org> wrote:
> rsmith added inline comments.
>
Author: bruno
Date: Thu Dec 8 20:22:47 2016
New Revision: 289174
URL: http://llvm.org/viewvc/llvm-project?rev=289174&view=rev
Log:
[CrashReproducer] Rewrite relative include paths
When -fmodules is on, the reproducer invocation currently leave paths
for include-like flags as is. If the path is r
+Eric, Marshall
I haven't looked, but: from the test name, "copy.fail.cpp", I suspect there is
a bug/incompatibility in the test. It likely relies on the compiler trying
(and failing) to copy something in a context where r288866 guarantees that
there is no copy.
> On 2016-Dec-08, at 18:00, Ad
Author: bruno
Date: Thu Dec 8 21:11:48 2016
New Revision: 289176
URL: http://llvm.org/viewvc/llvm-project?rev=289176&view=rev
Log:
[CrashReproducer] Provide a clean dir path for -fmodules-cache-path
The most common workflow with module reproducers involves deleting the
module cache before runnin
Author: ericwf
Date: Thu Dec 8 21:17:25 2016
New Revision: 289177
URL: http://llvm.org/viewvc/llvm-project?rev=289177&view=rev
Log:
Turn off header sorting and comment formatting
Modified:
libcxx/trunk/.clang-format
Modified: libcxx/trunk/.clang-format
URL:
http://llvm.org/viewvc/llvm-proj
Author: ericwf
Date: Thu Dec 8 21:18:45 2016
New Revision: 289178
URL: http://llvm.org/viewvc/llvm-project?rev=289178&view=rev
Log:
fix misspelling in .clang-format
Modified:
libcxx/trunk/.clang-format
Modified: libcxx/trunk/.clang-format
URL:
http://llvm.org/viewvc/llvm-project/libcxx/tru
On 8 Dec 2016 7:17 pm, "Duncan P. N. Exon Smith via cfe-commits" <
cfe-commits@lists.llvm.org> wrote:
+Eric, Marshall
I haven't looked, but: from the test name, "copy.fail.cpp", I suspect there
is a bug/incompatibility in the test. It likely relies on the compiler
trying (and failing) to copy so
Hmm, never mind. The test seems to preclude copy elision if I'm looking in the
right place:
std::streambuf &get();
int main()
{
std::streambuf sb = get(); // expected-error
}
> On 2016-Dec-08, at 19:17, Duncan P. N. Exon Smith via cfe-commits
> wrote:
>
> +Eric, Marshall
>
> I haven't
> On 2016-Dec-08, at 19:29, Richard Smith wrote:
>
> On 8 Dec 2016 7:17 pm, "Duncan P. N. Exon Smith via cfe-commits"
> wrote:
> +Eric, Marshall
>
> I haven't looked, but: from the test name, "copy.fail.cpp", I suspect there
> is a bug/incompatibility in the test. It likely relies on the co
Author: bruno
Date: Thu Dec 8 21:30:46 2016
New Revision: 289181
URL: http://llvm.org/viewvc/llvm-project?rev=289181&view=rev
Log:
[Headers] Add #include_next for tgmath.h on Darwin
Allow darwin to provide additional definitions and implementation
specifc values for tgmath.h on Apple platforms.
96 matches
Mail list logo
