vlad.tsyrklevich added a comment.
Hello @NoQ, I just wanted to ping you on the updated change since I'm not sure
if you saw it.
https://reviews.llvm.org/D28445
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/ma
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
https://reviews.llvm.org/D29143
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
mgorny updated this revision to Diff 85875.
mgorny retitled this revision from "[Driver] Pass -lunwind when using libc++ +
compiler-rt on Linux" to "[Driver] Pass -lunwind along with compiler-rt when
necessary on Linux".
mgorny edited the summary of this revision.
mgorny added a comment.
Herald a
echuraev added a comment.
This diagnostic was added in this commit: https://reviews.llvm.org/rL289979
It is something like that: "use of undeclared identifier 'double2'; did you
mean 'double'?"
This message isn't clear enough and it is difficult to understand that I forgot
to enable cl_khr_fp64
Author: asiri
Date: Thu Jan 26 04:38:03 2017
New Revision: 293166
URL: http://llvm.org/viewvc/llvm-project?rev=293166&view=rev
Log:
Fix chromium build (libcxxabi)
Pull the dependency on pthread_mach_thread_np() back into libcxxabi.
Modified:
libcxxabi/trunk/src/cxa_guard.cpp
Modified: libcx
Author: asiri
Date: Thu Jan 26 04:40:17 2017
New Revision: 293167
URL: http://llvm.org/viewvc/llvm-project?rev=293167&view=rev
Log:
Fix chromium build (libcxx)
Remove the reference to pthread_mach_thread_np() in libcxx headers.
Modified:
libcxx/trunk/include/__threading_support
Modified: li
Anastasia updated this revision to Diff 85878.
Anastasia edited reviewers, added: arsenm; removed: pekka.jaaskelainen.
Anastasia removed a subscriber: arsenm.
Anastasia added a comment.
Herald added a subscriber: wdng.
Added other CL versions to testing (from comment by Matt)!
https://reviews.ll
Anastasia marked 2 inline comments as done.
Anastasia added inline comments.
Comment at: test/SemaOpenCL/logical-ops.cl:1-3
+// RUN: %clang_cc1 %s -verify -cl-std=CL1.1 -triple x86_64-unknown-linux-gnu
+// RUN: %clang_cc1 %s -verify -cl-std=CL1.2 -triple x86_64-unknown-linux-gnu
sbarzowski updated this revision to Diff 85882.
sbarzowski marked 3 inline comments as done.
sbarzowski added a comment.
Improved messages, added tests with templates, fixed some typos.
https://reviews.llvm.org/D19201
Files:
clang-tidy/misc/CMakeLists.txt
clang-tidy/misc/MiscTidyModule.cpp
Hi Nico,
Hopefully I've sorted this out in r293166/r293167. Please let me know if
things are not back to normal.
(I'll keep an eye on your builder too)
/ Asiri
On Wed, Jan 25, 2017 at 8:42 PM, Asiri Rathnayake <
asiri.rathnay...@gmail.com> wrote:
> Hi Nico,
>
> Thanks for the links. I may have
Prazek added a comment.
In https://reviews.llvm.org/D29151#656887, @Eugene.Zelenko wrote:
> General question: isn't Static Analyzer is better place for such workflow
> checks?
Probably yes, but it is much harder to implement this there. Other problem is
that it would be probably a part
of on
Anastasia updated this revision to Diff 85888.
Anastasia added a comment.
Updated NULL ptr generation and added separate CodeGen test for checking
amended Block generation behaviour in OpenCL!
https://reviews.llvm.org/D28814
Files:
lib/AST/Expr.cpp
lib/CodeGen/CGBlocks.cpp
lib/CodeGen/CG
Anastasia marked 2 inline comments as done.
Anastasia added inline comments.
Comment at: test/CodeGenOpenCL/cl20-device-side-enqueue.cl:3
// RUN: %clang_cc1 %s -cl-std=CL2.0 -ffake-address-space-map -O0 -emit-llvm -o
- -triple "spir64-unknown-unknown" | FileCheck %s --check-pre
Anastasia updated this revision to Diff 85890.
Anastasia marked an inline comment as done.
Anastasia retitled this revision from "[OpenCL] Accept logical NOT for pointer
types in CL1.0 and CL1.1" to "[OpenCL] Accept logical NOT for pointer types in
CL1.1".
https://reviews.llvm.org/D29038
Files:
Anastasia added inline comments.
Comment at: test/SemaOpenCL/logical-ops.cl:1-3
+// RUN: %clang_cc1 %s -verify -cl-std=CL1.1 -triple x86_64-unknown-linux-gnu
+// RUN: %clang_cc1 %s -verify -cl-std=CL1.2 -triple x86_64-unknown-linux-gnu
+
Anastasia wrote:
> arsenm
On Wed, Jan 25, 2017 at 9:11 AM, Hans Wennborg wrote:
> Should we merge this to 4.0?
>
Yes, please.
-- Marshall
>
> On Tue, Jan 24, 2017 at 3:09 PM, Marshall Clow via cfe-commits
> wrote:
> > Author: marshall
> > Date: Tue Jan 24 17:09:12 2017
> > New Revision: 292990
> >
> > URL: http://llv
On Wed, Jan 25, 2017 at 10:58 PM, Marshall Clow via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: marshall
> Date: Thu Jan 26 00:58:29 2017
> New Revision: 293154
>
> URL: http://llvm.org/viewvc/llvm-project?rev=293154&view=rev
> Log:
> Use the new __has_feature(cxx_constexpr_string_b
ioeric created this revision.
For example, when we change 'na' to "nb::nc", we need to add leading '::' to
references "::nc::X" in the changed namespace.
https://reviews.llvm.org/D29176
Files:
change-namespace/ChangeNamespace.cpp
unittests/change-namespace/ChangeNamespaceTests.cpp
Index: u
delesley added a comment.
This looks good to me.
https://reviews.llvm.org/D28520
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: marshall
Date: Thu Jan 26 08:36:14 2017
New Revision: 293179
URL: http://llvm.org/viewvc/llvm-project?rev=293179&view=rev
Log:
Fixed a couple of invalid statuses for 2665 and 2758
Modified:
libcxx/trunk/www/cxx1z_status.html
Modified: libcxx/trunk/www/cxx1z_status.html
URL:
http://l
davidb added a comment.
ping
Repository:
rL LLVM
https://reviews.llvm.org/D27985
___
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.
LGTM! Thank you for being patient while we figured this out. :-)
https://reviews.llvm.org/D28520
___
cfe-commits mailing list
cfe-c
yaxunl accepted this revision.
yaxunl added a comment.
This revision is now accepted and ready to land.
LGTM. Thanks!
https://reviews.llvm.org/D28814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.
Only nits below.
Comment at: change-namespace/ChangeNamespace.cpp:232
+auto &DeclNsTop = DeclNsSplitted.front();
+for (auto &Ns : NsNameSplitted)
+ if (Ns == D
ioeric updated this revision to Diff 85903.
ioeric marked 3 inline comments as done.
ioeric added a comment.
- Addressed comments.
https://reviews.llvm.org/D29176
Files:
change-namespace/ChangeNamespace.cpp
unittests/change-namespace/ChangeNamespaceTests.cpp
Index: unittests/change-namespa
Author: ioeric
Date: Thu Jan 26 09:08:44 2017
New Revision: 293182
URL: http://llvm.org/viewvc/llvm-project?rev=293182&view=rev
Log:
[change-namespace] add leading '::' to references in new namespace when name
conflict is possible.
Summary:
For example, when we change 'na' to "nb::nc", we need t
This revision was automatically updated to reflect the committed changes.
Closed by commit rL293182: [change-namespace] add leading '::' to references in
new namespace when name… (authored by ioeric).
Changed prior to commit:
https://reviews.llvm.org/D29176?vs=85903&id=85904#toc
Repository:
yaxunl added a comment.
with the new pragma for associating types with extensions. This feature can be
implemented by solely modify opencl-c.h by adding #pragma OPENCL EXTENSION
cl_khr_fp64 : begin/end around vector double type definitions, e.g.
#pragma OPENCL EXTENSION cl_khr_fp64 : begin
This revision was automatically updated to reflect the committed changes.
Closed by commit rL293183: [OpenMP] Codegen support for 'target teams' on the
NVPTX device. (authored by arpith).
Changed prior to commit:
https://reviews.llvm.org/D29143?vs=85791&id=85913#toc
Repository:
rL LLVM
http
Author: arpith
Date: Thu Jan 26 09:43:27 2017
New Revision: 293183
URL: http://llvm.org/viewvc/llvm-project?rev=293183&view=rev
Log:
[OpenMP] Codegen support for 'target teams' on the NVPTX device.
This is a simple patch to teach OpenMP codegen to emit the construct
in Generic mode.
Reviewers: A
ioeric created this revision.
https://reviews.llvm.org/D29182
Files:
change-namespace/ChangeNamespace.cpp
unittests/change-namespace/ChangeNamespaceTests.cpp
Index: unittests/change-namespace/ChangeNamespaceTests.cpp
===
--- un
bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.
lg
https://reviews.llvm.org/D29182
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
Author: ioeric
Date: Thu Jan 26 10:31:32 2017
New Revision: 293187
URL: http://llvm.org/viewvc/llvm-project?rev=293187&view=rev
Log:
[change-namespace] correctly shorten namespace when references have leading '::'
Reviewers: bkramer
Subscribers: cfe-commits
Differential Revision: https://review
This revision was automatically updated to reflect the committed changes.
Closed by commit rL293187: [change-namespace] correctly shorten namespace when
references have leading '::' (authored by ioeric).
Changed prior to commit:
https://reviews.llvm.org/D29182?vs=85917&id=85919#toc
Repository:
emaste added a comment.
> My concern is that the stock gcc distributions for FreeBSD, NetBSD from your
> sources doesn't define __mips_abicalls,
I think we should consider that a bug in GCC that we'll rectify. If
`__mips_abicalls` is used in Linux GCC then we can expect 3rd party sources to
ch
baloghadamsoftware created this revision.
Take the following example:
class Int {
public:
Int(int n = 0): num(n) {}
Int(const Int& rhs): num(rhs.num) {}
Int& operator=(const Int& rhs) { num = rhs.num; }
operator int() { return num; }
private:
int num;
};
templa
xazax.hun added a comment.
Great find! Could you transform your examole into a testcasr that fails before
this patch? I think there should be already some tests for call graphs that
you can take a look at.
https://reviews.llvm.org/D29183
___
cfe-
baloghadamsoftware added a comment.
In https://reviews.llvm.org/D29183#657669, @xazax.hun wrote:
> Great find! Could you transform your examole into a testcasr that fails
> before this patch? I think there should be already some tests for call
> graphs that you can take a look at.
Yes, but I
baloghadamsoftware added a comment.
Some comments how I found this bug. I got some strange false positives even
after the fix for the iterator past end checker
(https://reviews.llvm.org/D28771). I could reproduce it with the following code:
#include
using std::list;
using std::prev;
faisalv updated this revision to Diff 85927.
faisalv added a comment.
I tried a different approach, not because I was convinced it was better, but
because it seemed (at the time) a simpler tweak - I'm not sure it is robust
enough - but would appreciate your thoughts on it.
The approach is predi
NoQ added a comment.
Hmm, we should have done better work reviewing https://reviews.llvm.org/D28905.
https://reviews.llvm.org/D29183
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
baloghadamsoftware added a comment.
I wonder if this could also be a problem for nested messages in Objective-C,
since the call for VisitChildren() is also missing from
VisitObjCMessageExpr()...
https://reviews.llvm.org/D29183
___
cfe-commits mail
baloghadamsoftware added a comment.
In https://reviews.llvm.org/D29183#657706, @NoQ wrote:
> Hmm, we should have done better work reviewing
> https://reviews.llvm.org/D28905.
Oh, someone was quicker than me by one week...
https://reviews.llvm.org/D29183
___
mprobst created this revision.
Herald added a subscriber: klimek.
The MPEG transport stream file format also uses ".ts" as its file extension.
This change detects its specific framing format (0x47 every 189 bytes) and
simply ignores MPEG TS files.
https://reviews.llvm.org/D29186
Files:
tools/
Yes, things are happy again, thanks!
Our bots currently use the macOS 10.10 SDK.
On Thu, Jan 26, 2017 at 6:56 AM, Asiri Rathnayake <
asiri.rathnay...@gmail.com> wrote:
> Hi Nico,
>
> Hopefully I've sorted this out in r293166/r293167. Please let me know if
> things are not back to normal.
>
> (I'
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
LGTM!
https://reviews.llvm.org/D28905
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
djasper accepted this revision.
djasper added inline comments.
This revision is now accepted and ready to land.
Comment at: unittests/Format/FormatTestJS.cpp:1002
+TEST_F(FormatTestJS, IgnoresMpegTS) {
+ char mpegTS[200];
+ mpegTS[0] = 0x47;
nit: Should be Mpeg
Author: hans
Date: Thu Jan 26 11:51:25 2017
New Revision: 293192
URL: http://llvm.org/viewvc/llvm-project?rev=293192&view=rev
Log:
Merging r292990:
r292990 | marshall | 2017-01-24 15:09:12 -0800 (Tue, 24 Jan 2017) | 1 line
C
On Thu, Jan 26, 2017 at 6:05 AM, Marshall Clow wrote:
> On Wed, Jan 25, 2017 at 9:11 AM, Hans Wennborg wrote:
>>
>> Should we merge this to 4.0?
>
>
> Yes, please.
r293192.
Cheers,
Hans
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://li
Author: hans
Date: Thu Jan 26 11:55:46 2017
New Revision: 293193
URL: http://llvm.org/viewvc/llvm-project?rev=293193&view=rev
Log:
Merging r293154:
r293154 | marshall | 2017-01-25 22:58:29 -0800 (Wed, 25 Jan 2017) | 1 line
U
On Thu, Jan 26, 2017 at 6:08 AM, Marshall Clow wrote:
> On Wed, Jan 25, 2017 at 10:58 PM, Marshall Clow via cfe-commits
> wrote:
>>
>> Author: marshall
>> Date: Thu Jan 26 00:58:29 2017
>> New Revision: 293154
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=293154&view=rev
>> Log:
>> Use the n
Michał suggested on the PR that this should be merged to the release branch.
Saleem, what do you think?
On Sat, Jan 21, 2017 at 8:22 AM, Saleem Abdulrasool via cfe-commits
wrote:
> Author: compnerd
> Date: Sat Jan 21 10:22:59 2017
> New Revision: 292723
>
> URL: http://llvm.org/viewvc/llvm-proje
mprobst updated this revision to Diff 85934.
mprobst added a comment.
- Move MPEG check into Format.cpp.
https://reviews.llvm.org/D29186
Files:
lib/Format/Format.cpp
tools/clang-format/ClangFormat.cpp
unittests/Format/FormatTestJS.cpp
Index: unittests/Format/FormatTestJS.cpp
===
mprobst added a comment.
PTAL, moved the check down a bit.
https://reviews.llvm.org/D29186
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Ping?
On Mon, Jan 23, 2017 at 4:27 PM, Hans Wennborg wrote:
> Ping?
>
> On Tue, Jan 17, 2017 at 4:16 PM, Hans Wennborg wrote:
>> Richard, what do you think?
>>
>> On Fri, Jan 13, 2017 at 3:16 PM, Eric Fiselier wrote:
>>> I would love to see this merged. It would make it easier to write libc++
>
What's the status here? Waiting for Marshall?
On Mon, Jan 23, 2017 at 10:20 AM, Hans Wennborg wrote:
> Sounds good to me.
>
> On Fri, Jan 20, 2017 at 11:38 AM, Eric Fiselier wrote:
>> We should merge this patch into the 4.0 release branch. It is needed to make
>> "check-all" pass when using GCC
On Fri, Jan 20, 2017 at 1:52 PM, Renato Golin wrote:
> On 20 January 2017 at 19:46, Eric Fiselier wrote:
>> This patch fixes a libc++abi test failure when compiled with GCC 5, 6, or 7.
>> We should merge this into 4.0 to help get `check-all` clean.
>
> Hi Eric,
>
> All good on my side, pass with
Author: ahatanak
Date: Thu Jan 26 12:13:06 2017
New Revision: 293194
URL: http://llvm.org/viewvc/llvm-project?rev=293194&view=rev
Log:
[Sema][ObjC] Make sure -Wblock-capture-autoreleasing issues a warning
even in the presence of nullability qualifiers.
This commit fixes bugs in r285031 where -Wbl
Eugene.Zelenko added a comment.
In https://reviews.llvm.org/D29151#657435, @Prazek wrote:
> In https://reviews.llvm.org/D29151#656887, @Eugene.Zelenko wrote:
>
> > General question: isn't Static Analyzer is better place for such workflow
> > checks?
>
>
> Probably yes, but it is much harder to i
JDevlieghere updated this revision to Diff 85937.
JDevlieghere added a comment.
- Added missing instantiations in test
Repository:
rL LLVM
https://reviews.llvm.org/D28768
Files:
clang-tidy/modernize/CMakeLists.txt
clang-tidy/modernize/ModernizeTidyModule.cpp
clang-tidy/modernize/Return
Author: dim
Date: Thu Jan 26 12:37:18 2017
New Revision: 293197
URL: http://llvm.org/viewvc/llvm-project?rev=293197&view=rev
Log:
Disable thread safety analysis for some functions in __thread_support
Many thread-related libc++ test cases fail on FreeBSD, due to the
following -Werror warnings:
This revision was automatically updated to reflect the committed changes.
Closed by commit rL293197: Disable thread safety analysis for some functions in
__thread_support (authored by dim).
Changed prior to commit:
https://reviews.llvm.org/D28520?vs=85794&id=85939#toc
Repository:
rL LLVM
ht
Author: ahatanak
Date: Thu Jan 26 12:51:10 2017
New Revision: 293199
URL: http://llvm.org/viewvc/llvm-project?rev=293199&view=rev
Log:
Turn on -Wblock-capture-autoreleasing by default.
Turning on the warning by default helps the users as it's a common
mistake to capture out-parameters in a block
alexeagle added a comment.
confused, wasn't this committed last year? is this an extra check?
https://reviews.llvm.org/D29186
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: rampitec
Date: Thu Jan 26 10:49:21 2017
New Revision: 293190
URL: http://llvm.org/viewvc/llvm-project?rev=293190&view=rev
Log:
Use TargetMachine adjustPassManager hook
Differential Revision: https://reviews.llvm.org/D28340
Modified:
cfe/trunk/lib/CodeGen/BackendUtil.cpp
Modified: cf
On 26 January 2017 at 18:22, Hans Wennborg wrote:
> What's the status here? Waiting for Marshall?
Possibly. I can confirm this made all the problem on ARM go away with
GCC 5.3 and 6.3.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists
Author: rsmith
Date: Thu Jan 26 14:40:47 2017
New Revision: 293207
URL: http://llvm.org/viewvc/llvm-project?rev=293207&view=rev
Log:
PR0091R3: Implement parsing support for using templates as types.
This change adds a new type node, DeducedTemplateSpecializationType, to
represent a type template
Author: pcc
Date: Thu Jan 26 15:09:48 2017
New Revision: 293209
URL: http://llvm.org/viewvc/llvm-project?rev=293209&view=rev
Log:
IRGen: When loading the main module in the distributed ThinLTO backend, look
for the module containing the summary.
Differential Revision: https://reviews.llvm.org/D2
This revision was automatically updated to reflect the committed changes.
Closed by commit rL293209: IRGen: When loading the main module in the
distributed ThinLTO backend, look… (authored by pcc).
Changed prior to commit:
https://reviews.llvm.org/D29067?vs=85618&id=85958#toc
Repository:
rL
george.burgess.iv added a comment.
Pinging early because this is a release blocker. :)
https://reviews.llvm.org/D28889
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
danalbert created this revision.
num_put::put uses %p for pointer types, but the exact format of %p is
implementation defined behavior for the C library. Compare output to
snprintf for portability.
Repository:
rL LLVM
https://reviews.llvm.org/D29197
Files:
test/std/localization/locale.cat
Author: pcc
Date: Thu Jan 26 15:38:48 2017
New Revision: 293210
URL: http://llvm.org/viewvc/llvm-project?rev=293210&view=rev
Log:
Add missing x86 requirement.
Modified:
cfe/trunk/test/CodeGen/thinlto-multi-module.ll
Modified: cfe/trunk/test/CodeGen/thinlto-multi-module.ll
URL:
http://llvm.o
hans created this revision.
Both on Mac and Windows, it's common to have a 'Users' directory in the root of
the filesystem, so one might specify a filename as '/Users/me/myfile.c'.
clang-cl (as well as MSVC's cl.exe) will interpret that as invoking '/U'
option, which is probably not what the us
On Mon, Jan 23, 2017 at 5:55 PM, Nico Weber wrote:
> On Mon, Jan 23, 2017 at 5:29 PM, Aaron Ballman
> wrote:
>>
>> On Mon, Jan 23, 2017 at 5:00 PM, Nico Weber via cfe-commits
>> wrote:
>> > On Sun, Jan 22, 2017 at 6:17 AM, Malcolm Parsons
>> >
>> > wrote:
>> >>
>> >> On 20 January 2017 at 21:32
jbcoe updated this revision to Diff 85967.
jbcoe marked 2 inline comments as done.
jbcoe added a comment.
Responses to some change requests. Input needed on template argument matcher.
https://reviews.llvm.org/D29118
Files:
clang-tools-extra/clang-tidy/CMakeLists.txt
clang-tools-extra/clang-
Hi,
I'm not sure why we only see this now, but this change is breaking
llvm bootstrap with -Werror:
http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/732/steps/bootstrap%20clang/logs/stdio
llvm/projects/libunwind/src/config.h:90:41: error: token pasting of
',' and __VA_ARGS__ is a G
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
Yes, this LGTM as well. Thank you for working on this!
https://reviews.llvm.org/D20693
___
cfe-commits mailing list
cfe-commits@lis
hintonda added a comment.
Great, thanks. Could you commit for me?
https://reviews.llvm.org/D20693
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Actually, the bot has been red since Jan 21 with this exact error.
On Thu, Jan 26, 2017 at 2:27 PM, Evgenii Stepanov
wrote:
> Hi,
>
> I'm not sure why we only see this now, but this change is breaking
> llvm bootstrap with -Werror:
> http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds
jbcoe added inline comments.
Comment at: clang-tools-extra/clang-tidy/safety/NoVectorBoolCheck.cpp:50
+diag(MatchedDecl->getLocation(),
+ " function %0 returns an instance of std::vector")
+<< MatchedDecl;
djehuti wrote:
> JonasToth wrote:
> >
amccarth added inline comments.
Comment at: include/clang/Basic/DiagnosticDriverKinds.td:252
+ InGroup>;
+def note_use_dashdash : Note<"Use '--' to treat subsequent arguments as
filenames">;
+
For Windows, wouldn't it make more sense to suggest using valid path
Author: aaronballman
Date: Thu Jan 26 16:34:24 2017
New Revision: 293217
URL: http://llvm.org/viewvc/llvm-project?rev=293217&view=rev
Log:
Implement a new clang-tidy check that suggests users replace dynamic exception
specifications with noexcept exception specifications.
Patch by Don Hinton.
A
Author: aaronballman
Date: Thu Jan 26 16:39:01 2017
New Revision: 293218
URL: http://llvm.org/viewvc/llvm-project?rev=293218&view=rev
Log:
Correcting a typo in the test case to appease bots.
Modified:
clang-tools-extra/trunk/test/clang-tidy/modernize-use-noexcept.cpp
Modified: clang-tools-ex
Author: rsmith
Date: Thu Jan 26 16:39:55 2017
New Revision: 293219
URL: http://llvm.org/viewvc/llvm-project?rev=293219&view=rev
Log:
[modules] When reading / writing a typedef that is a name for linkage for
another declaration, ensure we actually serialize / deserialize that
declaration.
Before t
aaron.ballman added a comment.
In https://reviews.llvm.org/D20693#658087, @hintonda wrote:
> Great, thanks. Could you commit for me?
Certainly! I've commit in r293217.
https://reviews.llvm.org/D20693
___
cfe-commits mailing list
cfe-commits@list
eugenis accepted this revision.
eugenis added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rL LLVM
https://reviews.llvm.org/D29077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-b
hans added inline comments.
Comment at: include/clang/Basic/DiagnosticDriverKinds.td:252
+ InGroup>;
+def note_use_dashdash : Note<"Use '--' to treat subsequent arguments as
filenames">;
+
amccarth wrote:
> For Windows, wouldn't it make more sense to suggest us
Author: rsmith
Date: Thu Jan 26 17:06:10 2017
New Revision: 293222
URL: http://llvm.org/viewvc/llvm-project?rev=293222&view=rev
Log:
Add files forgotten in r293219.
Added:
cfe/trunk/test/Modules/Inputs/merge-name-for-linkage/c1.h
cfe/trunk/test/Modules/Inputs/merge-name-for-linkage/c2.h
Author: rsmith
Date: Thu Jan 26 17:07:59 2017
New Revision: 293223
URL: http://llvm.org/viewvc/llvm-project?rev=293223&view=rev
Log:
[modules] Additional tests.
Added:
cfe/trunk/test/Modules/Inputs/anon-redecl/
cfe/trunk/test/Modules/Inputs/anon-redecl/a.h
cfe/trunk/test/Modules/Input
Should we merge this to the release branch?
On Fri, Jan 20, 2017 at 12:57 AM, Alexey Bataev via cfe-commits
wrote:
> Author: abataev
> Date: Fri Jan 20 02:57:28 2017
> New Revision: 292590
>
> URL: http://llvm.org/viewvc/llvm-project?rev=292590&view=rev
> Log:
> [OPENMP] Fix for PR31643: Clang cr
Hi Richard,
I don't know if you have noticed, but the test you added in this commit is
failing on both the Linux and Windows PS4 bots. The test is failing with an
assertion failure:
Assertion failed: !A->getDeducedType().isNull() && "cannot request the size of
an undeduced or dependent auto ty
Should we merge this to the release branch?
On Mon, Jan 16, 2017 at 8:14 PM, David Majnemer via cfe-commits
wrote:
> Author: majnemer
> Date: Mon Jan 16 22:14:25 2017
> New Revision: 292194
>
> URL: http://llvm.org/viewvc/llvm-project?rev=292194&view=rev
> Log:
> [AST] AttributedType should deriv
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
https://reviews.llvm.org/D29198
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
A candidate for clang 4?
On Thu, Jan 19, 2017 at 5:19 PM, Richard Smith via cfe-commits
wrote:
> Author: rsmith
> Date: Thu Jan 19 19:19:46 2017
> New Revision: 292561
>
> URL: http://llvm.org/viewvc/llvm-project?rev=292561&view=rev
> Log:
> PR31701: Fix crash on invalid caused by parsing a depen
Author: pcc
Date: Thu Jan 26 17:51:50 2017
New Revision: 293231
URL: http://llvm.org/viewvc/llvm-project?rev=293231&view=rev
Log:
Re-apply r292662, "IRGen: Start using the WriteThinLTOBitcode pass."
The internal build issue has been resolved.
Added:
cfe/trunk/test/CodeGenCXX/type-metadata-th
Author: eugenezelenko
Date: Thu Jan 26 17:58:21 2017
New Revision: 293234
URL: http://llvm.org/viewvc/llvm-project?rev=293234&view=rev
Log:
[Clang-tidy documentation] Consistency (fix-it); 80 characters per line.
Modified:
clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-no-m
danielcdh created this revision.
Herald added a subscriber: mehdi_amini.
cfe change for https://reviews.llvm.org/D29203
https://reviews.llvm.org/D29205
Files:
lib/CodeGen/BackendUtil.cpp
lib/CodeGen/CodeGenFunction.cpp
Index: lib/CodeGen/CodeGenFunction.cpp
===
Sure, why not.
On 26 January 2017 at 15:56, Hans Wennborg wrote:
> A candidate for clang 4?
>
> On Thu, Jan 19, 2017 at 5:19 PM, Richard Smith via cfe-commits
> wrote:
> > Author: rsmith
> > Date: Thu Jan 19 19:19:46 2017
> > New Revision: 292561
> >
> > URL: http://llvm.org/viewvc/llvm-project
Hello everyone,
LLVM buildmaster will be updated and restarted after 6 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
Author: rsmith
Date: Thu Jan 26 19:54:42 2017
New Revision: 293252
URL: http://llvm.org/viewvc/llvm-project?rev=293252&view=rev
Log:
Improve workaround for Sphinx's lack of support for command line options
containing '+', '.' etc. to be more stable as the set of options changes.
Modified:
cf
1 - 100 of 114 matches
Mail list logo
