aaboud created this revision.
aaboud added reviewers: dblaikie, rsmith, aprantl, probinson.
aaboud added a subscriber: cfe-commits.
Added support for FE Clang to create Debug Info entries (DIMacro and
DIMacroFile) into generated module if "debug-info-kind" flag is set to
"standalone".
http://re
aaboud added inline comments.
Comment at: include/clang/AST/ASTConsumer.h:163
@@ -155,1 +162,3 @@
+ /// The caller takes ownership on the returned pointer.
+ virtual std::unique_ptr
CreatePreprocessorCallbacks(Preprocessor &PP);
};
Richard,
I know that you sug
andreybokhanko added a comment.
In http://reviews.llvm.org/D15686#325266, @rnk wrote:
> I thought we already addressed this issue with @rjmccall and decided that, if
> the user intentionally declares extern "C" variables with an _Z prefix, then
> we know they are intentionally attempting to nam
xazax.hun updated this revision to Diff 44721.
xazax.hun added a comment.
- Moved the API to CallEvent.
http://reviews.llvm.org/D15921
Files:
include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h
lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp
lib/StaticAnalyzer/Core/CallEvent.c
xazax.hun marked an inline comment as done.
Comment at: include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h:312
@@ +311,3 @@
+ /// calls.
+ bool isCalled(const CallEvent &Call, const CallDescription &CD);
+
zaks.anna wrote:
> The API is a bit awkwar
andreybokhanko created this revision.
andreybokhanko added reviewers: rnk, mkuper.
andreybokhanko added a subscriber: cfe-commits.
Correct setting of UserLabelPrefix for MCU target
http://reviews.llvm.org/D16138
Files:
lib/Basic/Targets.cpp
test/Preprocessor/elfiamcu-predefines.c
Index: lib
andreybokhanko updated this revision to Diff 44726.
andreybokhanko added a comment.
Patch updated to be in line with llvm trunk. Sorry for the noise.
http://reviews.llvm.org/D16138
Files:
lib/Basic/Targets.cpp
test/Preprocessor/elfiamcu-predefines.c
Index: lib/Basic/Targets.cpp
===
alexfh added a comment.
Thank you for the patch! I have a few concerns though. See inline comments.
Comment at: clang-tidy/ClangTidy.cpp:348
@@ +347,3 @@
+ LocalName.str() };
+ for (const auto &Name : Names) {
+auto Iter = CheckOptions.fi
alexfh added inline comments.
Comment at: clang-tidy/ClangTidyOptions.h:216
@@ +215,3 @@
+/// HeaderFileExtensions.
+bool endWithHeaderFileExtensions(llvm::StringRef FileName,
+ llvm::StringRef HeaderFileExtensions);
hokein wrote:
>
Author: abataev
Date: Wed Jan 13 05:18:54 2016
New Revision: 257586
URL: http://llvm.org/viewvc/llvm-project?rev=257586&view=rev
Log:
[OPENMP] Fix for declarative/standalone directives use.
Fixes processing of declarative directives and standalone executable
directives. Declarative directives sho
Anastasia added inline comments.
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:593
@@ -592,2 +592,3 @@
+def err_no_declarators : Error<"declaration does not declare anything">;
def ext_typedef_without_a_name : ExtWarn<"typedef requires a name">,
InGroup;
-
Anastasia added inline comments.
Comment at: test/SemaOpenCL/invalid-pipe-builtin-cl2.0.cl:37
@@ +36,3 @@
+ write_pipe(p, tmp); // expected-error {{invalid argument type to function
write_pipe (expecting: 'int *')}}
+ read_pipe(p, ptr);// expected-error {{invalid pipe acc
NoQ updated this revision to Diff 44734.
NoQ added a comment.
Rebase on top of http://reviews.llvm.org/D12901 - support `SymbolCast` in the
explainer, as it finally appears in the wild.
http://reviews.llvm.org/D15448
Files:
docs/analyzer/DebugChecks.rst
include/clang/StaticAnalyzer/Checker
Hi,
On Wed, Jan 13, 2016 at 12:38 AM, Laszlo Nagy via cfe-commits
wrote:
> Author: rizsotto
> Date: Tue Jan 12 16:38:41 2016
> New Revision: 257533
>
> URL: http://llvm.org/viewvc/llvm-project?rev=257533&view=rev
> Log:
> D9600: Add scan-build python implementation
This doesn't seem to be instal
NoQ added a comment.
Nope, will commit without `SymbolCast` support for now, encountered some issues
with http://reviews.llvm.org/D12901 that would probably be worth a separate
commit.
http://reviews.llvm.org/D15448
___
cfe-commits mailing list
cf
bhushan created this revision.
bhushan added a reviewer: dsanders.
bhushan added subscribers: cfe-commits, jaydeep, mohit.bhakkad, nitesh.jain,
sagar.
bhushan set the repository for this revision to rL LLVM.
Herald added a reviewer: vkalintiris.
This patch sets CPU string to its default value whe
xazax.hun added inline comments.
Comment at: tools/clang/lib/StaticAnalyzer/Core/MemRegion.cpp:653
@@ +652,3 @@
+llvm::SmallString<2> intValAsString;
+IndexInArray.toString(intValAsString);
+std::string idx{intValAsString.begin(), intValAsString.end()};
---
This revision was automatically updated to reflect the committed changes.
Closed by commit rL257598: [analyzer] Fix SVal/SymExpr/MemRegion class and enum
names for consistency. (authored by dergachev).
Changed prior to commit:
http://reviews.llvm.org/D16062?vs=44628&id=44745#toc
Repository:
Author: dergachev
Date: Wed Jan 13 07:49:29 2016
New Revision: 257598
URL: http://llvm.org/viewvc/llvm-project?rev=257598&view=rev
Log:
[analyzer] Fix SVal/SymExpr/MemRegion class and enum names for consistency.
The purpose of these changes is to simplify introduction of definition files
for the
rafael added a subscriber: rafael.
rafael accepted this revision.
rafael added a reviewer: rafael.
rafael added a comment.
This revision is now accepted and ready to land.
LGTM, but could you change the default in a followup commit? TargetInfo should
really be setting it to "" since it is far mor
alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.
Looks good with one nit. I'll update the comment myself and commit the patch
for you.
Thank you for the contribution!
Comment at: clang-tidy/misc/VirtualNearMissCheck.cpp:60
Author: alexfh
Date: Wed Jan 13 08:16:35 2016
New Revision: 257599
URL: http://llvm.org/viewvc/llvm-project?rev=257599&view=rev
Log:
Support virtual-near-miss check.
Summary: Virtual function override near miss detection. Function complete. Test
complete. Do not conduct Fix for now.
Reviewers:
Author: alexfh
Date: Wed Jan 13 08:16:43 2016
New Revision: 257600
URL: http://llvm.org/viewvc/llvm-project?rev=257600&view=rev
Log:
[clang-tidy] Slightly clarified comments.
Modified:
clang-tools-extra/trunk/clang-tidy/misc/VirtualNearMissCheck.h
Modified: clang-tools-extra/trunk/clang-tidy
This revision was automatically updated to reflect the committed changes.
Closed by commit rL257599: Support virtual-near-miss check. (authored by
alexfh).
Changed prior to commit:
http://reviews.llvm.org/D15823?vs=44661&id=44746#toc
Repository:
rL LLVM
http://reviews.llvm.org/D15823
Files
Author: s.egerton
Date: Wed Jan 13 08:27:59 2016
New Revision: 257602
URL: http://llvm.org/viewvc/llvm-project?rev=257602&view=rev
Log:
[mips] Added support for -Wa,-mips32 and similar.
Reviewers: vkalintiris, dsanders
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D150
This revision was automatically updated to reflect the committed changes.
Closed by commit rL257605: [analyzer] Provide .def-files and visitors for
SVal/SymExpr/MemRegion. (authored by dergachev).
Changed prior to commit:
http://reviews.llvm.org/D15448?vs=44734&id=44752#toc
Repository:
rL LL
bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.
After pondering this patch for a while I couldn't come up with a case where
this would cause problems, but it provides great improvements for editing
headers in an interactive setup. So LGTM
Author: dergachev
Date: Wed Jan 13 09:13:48 2016
New Revision: 257605
URL: http://llvm.org/viewvc/llvm-project?rev=257605&view=rev
Log:
[analyzer] Provide .def-files and visitors for SVal/SymExpr/MemRegion.
Provide separate visitor templates for the three hierarchies, and also
the `FullSValVisito
Author: dergachev
Date: Wed Jan 13 09:52:25 2016
New Revision: 257608
URL: http://llvm.org/viewvc/llvm-project?rev=257608&view=rev
Log:
Revert "[analyzer] Provide .def-files and visitors for SVal/SymExpr/MemRegion."
This reverts commit r257605.
The test fails on architectures that use unsigned i
ABataev updated this revision to Diff 44754.
ABataev added a comment.
Update after review
http://reviews.llvm.org/D15709
Files:
include/clang/Basic/Attr.td
include/clang/Basic/AttrDocs.td
include/clang/Basic/DiagnosticSemaKinds.td
lib/CodeGen/TargetInfo.cpp
lib/Sema/SemaDeclAttr.cpp
alexfh added a comment.
Looks good with a couple of nits. I'll be happy to submit your patch once you
address the comments.
Also, please include more context in your patches. See
http://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface.
Comment at: clan
Author: s.egerton
Date: Wed Jan 13 10:19:33 2016
New Revision: 257610
URL: http://llvm.org/viewvc/llvm-project?rev=257610&view=rev
Log:
Revert r257602 because it breaks integrated-as.s on mips hosts.
Modified:
cfe/trunk/lib/Driver/Tools.cpp
cfe/trunk/test/Driver/mips-ias-Wa.s
Modified: c
EricWF added a comment.
Almost LGTM. Could you re-upload with more context quickly? (ex git diff -U999)
Comment at: include/iterator:1418
@@ +1417,3 @@
+template
+struct __libcpp_is_trivial_iterator >
+ : public
_LIBCPP_BOOL_CONSTANT(__libcpp_is_trivial_iterator<_Iter>::
Author: djg
Date: Wed Jan 13 10:32:00 2016
New Revision: 257612
URL: http://llvm.org/viewvc/llvm-project?rev=257612&view=rev
Log:
[WebAssembly] Set std::numeric_limits's traps field for WebAssembly.
WebAssembly's integer division instruction traps on division by zero; set the
traps field of integ
Author: djg
Date: Wed Jan 13 10:39:30 2016
New Revision: 257614
URL: http://llvm.org/viewvc/llvm-project?rev=257614&view=rev
Log:
[WebAssembly] Accomodate wasm's 128-bit long double.
Modified:
libcxxabi/trunk/src/cxa_demangle.cpp
Modified: libcxxabi/trunk/src/cxa_demangle.cpp
URL:
http://ll
Author: djasper
Date: Wed Jan 13 10:41:34 2016
New Revision: 257615
URL: http://llvm.org/viewvc/llvm-project?rev=257615&view=rev
Log:
clang-format: [ObjC+JS] Allow bin-packing of array literals.
After reading the style guides again, they don't actually say how to
pack or not pack array literals.
aprantl added a comment.
Added a few stylistic comments and request for more documentation.
Comment at: lib/CodeGen/MacroPPCallbacks.cpp:1
@@ +1,2 @@
+//===--- MacroPPCallbacks.h ---*- C++
-*-===//
+//
Please run your patch t
NoQ removed rL LLVM as the repository for this revision.
NoQ updated this revision to Diff 44758.
NoQ added a comment.
Reverted the patch due to a few issues. This revision should fix these issues.
The explain-svals test is fixed to target a specific target, in order to make
sure that the defini
jroelofs updated this revision to Diff 44759.
jroelofs added a comment.
clang-format the patch, and rename method.
http://reviews.llvm.org/D15528
Files:
clang-tidy/ClangTidy.cpp
clang-tidy/ClangTidy.h
clang-tidy/ClangTidyDiagnosticConsumer.cpp
clang-tidy/ClangTidyDiagnosticConsumer.h
Author: hans
Date: Wed Jan 13 11:33:21 2016
New Revision: 257629
URL: http://llvm.org/viewvc/llvm-project?rev=257629&view=rev
Log:
Update version to 3.9
Modified:
libcxx/trunk/include/__config
Modified: libcxx/trunk/include/__config
URL:
http://llvm.org/viewvc/llvm-project/libcxx/trunk/incl
Author: hans
Date: Wed Jan 13 11:32:59 2016
New Revision: 257628
URL: http://llvm.org/viewvc/llvm-project?rev=257628&view=rev
Log:
Update version to 3.9
Modified:
cfe/trunk/docs/ReleaseNotes.rst
cfe/trunk/docs/conf.py
Modified: cfe/trunk/docs/ReleaseNotes.rst
URL:
http://llvm.org/viewvc
Author: hans
Date: Wed Jan 13 11:35:10 2016
New Revision: 257635
URL: http://llvm.org/viewvc/llvm-project?rev=257635&view=rev
Log:
Creating release_38 branch off revision 257626
Added:
libcxxabi/branches/release_38/
- copied from r257626, libcxxabi/trunk/
__
Author: hans
Date: Wed Jan 13 11:35:07 2016
New Revision: 257634
URL: http://llvm.org/viewvc/llvm-project?rev=257634&view=rev
Log:
Creating release_38 branch off revision 257626
Added:
libcxx/branches/release_38/ (props changed)
- copied from r257626, libcxx/trunk/
Propchange: libcxx
Author: hans
Date: Wed Jan 13 11:35:29 2016
New Revision: 257641
URL: http://llvm.org/viewvc/llvm-project?rev=257641&view=rev
Log:
Creating release_38 branch off revision 257626
Added:
libunwind/branches/release_38/
- copied from r257626, libunwind/trunk/
__
Author: jroelofs
Date: Wed Jan 13 11:36:41 2016
New Revision: 257642
URL: http://llvm.org/viewvc/llvm-project?rev=257642&view=rev
Log:
Teach clang-tidy how to upgrade warnings into errors.
Similar in format to the `-checks=` argument, this new `-warnings-as-errors=`
argument upgrades any warnings
Marshall, Eric,
Please check that this was OK, as I don't usually commit to libc++.
Thanks,
Hans
On Wed, Jan 13, 2016 at 9:33 AM, Hans Wennborg via cfe-commits
wrote:
> Author: hans
> Date: Wed Jan 13 11:33:21 2016
> New Revision: 257629
>
> URL: http://llvm.org/viewvc/llvm-project?rev=257629&v
jroelofs accepted this revision.
jroelofs added a reviewer: jroelofs.
jroelofs marked 2 inline comments as done.
jroelofs added a comment.
Thanks for the review! Committed r257642.
http://reviews.llvm.org/D15528
___
cfe-commits mailing list
cfe-comm
tra added inline comments.
Comment at: test/Driver/cuda-preprocess.cu:13-16
@@ +12,6 @@
+
+// RUN: %clang -E -target x86_64-linux-gnu --cuda-gpu-arch=sm_20 %s 2>&1 \
+// RUN: | FileCheck -check-prefix NOARCH %s
+// RUN: %clang -E -target x86_64-linux-gnu --cuda-gpu-arch=sm_20
-
tra accepted this revision.
tra added a comment.
This revision is now accepted and ready to land.
LGTM.
http://reviews.llvm.org/D16081
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ahatanak added a comment.
Ping.
The same pragma is being used in other places (regex.cpp and locale.cpp) to
silence the warning.
http://reviews.llvm.org/D15897
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/m
hubert.reinterpretcast added inline comments.
Comment at: include/clang/AST/DeclTemplate.h:373
@@ -372,1 +372,3 @@
+ /// Whether this is a (C++ Concepts TS) function concept.
+ bool isConcept() const { return TemplatedDecl.getInt(); }
This code is not limited
NoQ added a reviewer: rsmith.
NoQ added a comment.
Richard: excuse me, adding you because you are an expert on the `modulemap`,
could you have a quick look at the proposed changes here and probably point me
in the right direction, because i'm not quite sure how to test the
modules-enabled build
mclow.lists added inline comments.
Comment at: include/iterator:1418
@@ +1417,3 @@
+template
+struct __libcpp_is_trivial_iterator >
+ : public
_LIBCPP_BOOL_CONSTANT(__libcpp_is_trivial_iterator<_Iter>::value) {};
EricWF wrote:
> Should this trait handle `c
Author: hans
Date: Wed Jan 13 13:14:03 2016
New Revision: 257652
URL: http://llvm.org/viewvc/llvm-project?rev=257652&view=rev
Log:
Update cxx_dr_status.html after the 3.8 branch
Modified:
cfe/trunk/www/cxx_dr_status.html
Modified: cfe/trunk/www/cxx_dr_status.html
URL:
http://llvm.org/viewvc
Richard, can you check that I got this right? Are there any more DRs
that were addressed in 3.8?
If it looks good, I'll merge this to the branch.
Cheers,
Hans
On Wed, Jan 13, 2016 at 11:14 AM, Hans Wennborg via cfe-commits
wrote:
> Author: hans
> Date: Wed Jan 13 13:14:03 2016
> New Revision: 2
jix created this revision.
jix added a subscriber: cfe-commits.
jix set the repository for this revision to rL LLVM.
Herald added a subscriber: aemerson.
Parameter default values are expected to be attached to all redeclarations of a
template. During serialization they are serialized only for the
On Fri, Jan 8, 2016 at 1:54 PM, Philippe Canal wrote:
> Hi Richard,
>
> In our use (Persistency of C++ objects, including the (meta)description of
> their data content as well as use the name as key to access the class'
> reflection information at run-time), the 'context' is indeed always the "at
xur marked an inline comment as done.
xur added a comment.
I'll sent the revised patch based on the comments shortly.
Comment at: lib/CodeGen/BackendUtil.cpp:435
@@ +434,3 @@
+ if (CodeGenOpts.ProfileIRInstr) {
+assert (!CodeGenOpts.ProfileInstrGenerate);
+if (!CodeGenO
xur updated this revision to Diff 44776.
xur added a comment.
added new tests and comments based on the review comments.
http://reviews.llvm.org/D15829
Files:
include/clang/Driver/Options.td
include/clang/Frontend/CodeGenOptions.def
lib/CodeGen/BackendUtil.cpp
lib/CodeGen/CodeGenModule.
Tests need to be updated, for example:
clang/test/Index/remap-load.c:7:11: error: expected string not found in input
// CHECK: remap-load.c:2:10: BinaryOperator= Extent=[2:10 - 2:23]
^
:334:1: note: scanning from here
// CHECK: remap-load.c:2:10: BinaryOperator=+ Extent=[2:10 - 2:23]
^
:
akyrtzi added a comment.
Tests need to be updated, for example:
clang/test/Index/remap-load.c:7:11: error: expected string not found in input
// CHECK: remap-load.c:2:10: BinaryOperator= Extent=[2:10 - 2:23]
^
:334:1: note: scanning from here
// CHECK: remap-load.c:2:10: BinaryOperator=+ Exte
jroelofs abandoned this revision.
jroelofs added a comment.
Hmm. There's no "close revision" button. Abandoning it in lieu of closing it.
http://reviews.llvm.org/D15528
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cg
comex added reviewers: aaron.ballman, rsmith.
comex added a comment.
Ping, and adding potential reviewers like I was supposed to do in the first
place.
http://reviews.llvm.org/D15907
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://li
ldionne added a reviewer: rsmith.
ldionne updated this revision to Diff 44779.
ldionne added a comment.
Rebase on top of the master branch, and add Richard Smith as a reviewer, since
he also reviewed David Majnemer's original patch (suggested by Nathan Wilson).
http://reviews.llvm.org/D15421
F
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
Sorry, this LGTM. You should push it to 3.8 as well.
Comment at: include/iterator:1418
@@ +1417,3 @@
+template
+struct __libcpp_is_trivial_iterator >
+ : public
_LIBCP
rsmith added a comment.
I think this functionality can be provided more simply by adding another flag
to `PrintingPolicy` to request fully-qualified names always be produced. It
already usually provides fully-qualified names; the only notable exception I
can see is that if a qualifier was alrea
sbenza created this revision.
sbenza added a reviewer: alexfh.
sbenza added a subscriber: cfe-commits.
Add check performance-faster-string-find.
It replaces single character string literals to character literals in calls to
string::find and friends.
http://reviews.llvm.org/D16152
Files:
clang
rsmith added inline comments.
Comment at: include/clang/Basic/Builtins.h:216-219
@@ -215,3 +215,6 @@
/// \brief This names the __make_integer_seq BuiltinTemplateDecl.
BTK__make_integer_seq
+
+ /// \brief This names the __nth_element BuiltinTemplateDecl.
+ , BTK__nth_elemen
rsmith added a comment.
Bikeshedding on the name a bit... how about `__type_pack_element`?
http://reviews.llvm.org/D15421
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jyknight created this revision.
jyknight added a subscriber: cfe-commits.
Herald added subscribers: jyknight, dschuff, jfb.
Fixes PR11517 for SPARC.
On most targets, clang lowers va_arg itself, eschewing the use of the
llvm vaarg instruction. This is necessary (at least for now) as the type
argum
Author: marshall
Date: Wed Jan 13 15:54:34 2016
New Revision: 257682
URL: http://llvm.org/viewvc/llvm-project?rev=257682&view=rev
Log:
Fix PR#25973 : 'basic_string::assign(InputIt, InputIt) doesn't provide the
strong exception safety guarantee'. This turned out to be a pervasive problem
in , whi
nwilson added a comment.
In http://reviews.llvm.org/D15421#326144, @rsmith wrote:
> Bikeshedding on the name a bit... how about `__type_pack_element`?
Hmm, I kind of felt like having `nth` in there implied we're indexing into
something... What about `__nth_pack_element`?
http://reviews.llvm.
ldionne added a comment.
In http://reviews.llvm.org/D15421#326209, @nwilson wrote:
> In http://reviews.llvm.org/D15421#326144, @rsmith wrote:
>
> > Bikeshedding on the name a bit... how about `__type_pack_element`?
>
>
> Hmm, I kind of felt like having `nth` in there implied we're indexing into
On Wed, Jan 13, 2016 at 2:31 PM, Nathan Wilson wrote:
> nwilson added a comment.
>
> In http://reviews.llvm.org/D15421#326144, @rsmith wrote:
>
> > Bikeshedding on the name a bit... how about `__type_pack_element`?
>
> Hmm, I kind of felt like having `nth` in there implied we're indexing into
> s
Apologies for reviving a long dead thread, but I think we need to re-evalute
this change.
With CMake 3.4 I’m seeing some nasty error spew on our internal builds. The
spew can be reproduced with the following CMake invocation:
> cmake -G Ninja -DLLVM_ENABLE_PIC=Off -DLLVM_INSTALL_TOOLCHAIN_ONLY=
The right way to update this file is to change the www/make_cxx_dr_status
script and then rerun it. (It expects to find a copy of the cwg_index.html
page from the wg21 website in the current directory.) Done in r257695.
On Wed, Jan 13, 2016 at 11:20 AM, Hans Wennborg wrote:
> Richard, can you ch
Hello,
In the attachment is a small patch for clang-format-sublime.py, that
prevents the console window from showing. Right now, it is flashing ever so
briefly, ever so annoyingly; this patches fixes that.
I've tested it on Windows and Mac OS X.
I do not have write access; if you agree with the
Author: rsmith
Date: Wed Jan 13 16:51:59 2016
New Revision: 257695
URL: http://llvm.org/viewvc/llvm-project?rev=257695&view=rev
Log:
Update make_cxx_dr_status after the 3.8 branch.
Modified:
cfe/trunk/www/make_cxx_dr_status
Modified: cfe/trunk/www/make_cxx_dr_status
URL:
http://llvm.org/vie
Author: marshall
Date: Wed Jan 13 16:52:36 2016
New Revision: 257696
URL: http://llvm.org/viewvc/llvm-project?rev=257696&view=rev
Log:
Fix test for C++03 - lacking noexcept
Modified:
libcxx/trunk/test/libcxx/strings/iterators.exceptions.pass.cpp
Modified: libcxx/trunk/test/libcxx/strings/ite
beanz created this revision.
beanz added reviewers: rnk, chapuni, hintonda.
beanz added a subscriber: cfe-commits.
CMake freaks out in trivial cases like:
> cmake -G Ninja -DLLVM_ENABLE_PIC=Off -DLLVM_INSTALL_TOOLCHAIN_ONLY=On ../llvm
This is a result of static ink dependencies not all being par
george.burgess.iv created this revision.
george.burgess.iv added reviewers: bob.wilson, rsmith.
george.burgess.iv added a subscriber: cfe-commits.
Fix PR26085 by suppressing diags from CheckSingleAssignmentConstraints during
overload resolution.
It seems that passing `Diagnose`/`Complain` flags
yes, it is just an initial drop. improvements are planed to make a full
replacement of the one written in Perl.
On Thu, Jan 14, 2016 at 12:14 AM, Ismail Donmez wrote:
> Hi,
>
> On Wed, Jan 13, 2016 at 12:38 AM, Laszlo Nagy via cfe-commits
> wrote:
> > Author: rizsotto
> > Date: Tue Jan 12 16:38
Author: marshall
Date: Wed Jan 13 17:05:15 2016
New Revision: 257702
URL: http://llvm.org/viewvc/llvm-project?rev=257702&view=rev
Log:
Better comments in test. NFC
Modified:
libcxx/trunk/test/libcxx/strings/iterators.noexcept.pass.cpp
Modified: libcxx/trunk/test/libcxx/strings/iterators.noex
On Wed, Jan 13, 2016 at 4:52 PM, Richard Smith
wrote:
> On Wed, Jan 13, 2016 at 2:31 PM, Nathan Wilson
> wrote:
>
>> nwilson added a comment.
>>
>> In http://reviews.llvm.org/D15421#326144, @rsmith wrote:
>>
>> > Bikeshedding on the name a bit... how about `__type_pack_element`?
>>
>> Hmm, I kin
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
LGTM
http://reviews.llvm.org/D16159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
Author: jroelofs
Date: Wed Jan 13 17:27:08 2016
New Revision: 257707
URL: http://llvm.org/viewvc/llvm-project?rev=257707&view=rev
Log:
Tame a -Wunknown-attributes warning
Modified:
libcxx/trunk/include/__config
Modified: libcxx/trunk/include/__config
URL:
http://llvm.org/viewvc/llvm-project
Author: gbiv
Date: Wed Jan 13 17:36:34 2016
New Revision: 257710
URL: http://llvm.org/viewvc/llvm-project?rev=257710&view=rev
Log:
[Sema] Suppress diags in overload resolution.
We were emitting diagnostics from our shiny new C-only overload
resolution mode. This patch attempts to silence all such
This revision was automatically updated to reflect the committed changes.
Closed by commit rL257710: [Sema] Suppress diags in overload resolution.
(authored by gbiv).
Changed prior to commit:
http://reviews.llvm.org/D16159?vs=44797&id=44803#toc
Repository:
rL LLVM
http://reviews.llvm.org/D1
Following Louis' suggestion, how about __pack_nth?
On Wed, Jan 13, 2016 at 3:16 PM, Nathan Wilson via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
>
>
> On Wed, Jan 13, 2016 at 4:52 PM, Richard Smith
> wrote:
>
>> On Wed, Jan 13, 2016 at 2:31 PM, Nathan Wilson
>> wrote:
>>
>>> nwilson added
eugenis updated this revision to Diff 44805.
eugenis marked 2 inline comments as done.
Repository:
rL LLVM
http://reviews.llvm.org/D15699
Files:
lib/CodeGen/CGClass.cpp
lib/CodeGen/CGExpr.cpp
lib/CodeGen/CodeGenFunction.h
lib/CodeGen/CodeGenModule.cpp
lib/Driver/Tools.cpp
test/Code
eugenis added inline comments.
Comment at: lib/CodeGen/CGExpr.cpp:2654-2678
@@ +2653,27 @@
+ for (auto CheckKindMaskPair : CheckKinds) {
+int Kind = CheckKindMaskPair.first;
+SanitizerMask Mask = CheckKindMaskPair.second;
+// All CFI checks are recoverable.
+asser
Thanks. I've merged this together with r257695 to 3.8 in r257714.
On Wed, Jan 13, 2016 at 2:55 PM, Richard Smith wrote:
> The right way to update this file is to change the www/make_cxx_dr_status
> script and then rerun it. (It expects to find a copy of the cwg_index.html
> page from the wg21 web
Author: hans
Date: Wed Jan 13 17:53:40 2016
New Revision: 257716
URL: http://llvm.org/viewvc/llvm-project?rev=257716&view=rev
Log:
Merging r257682:
r257682 | marshall | 2016-01-13 13:54:34 -0800 (Wed, 13 Jan 2016) | 1 line
F
FYI: This patch should to be merged into the clang 3.8.0 branch.
Thank you!
George
On Wed, Jan 13, 2016 at 3:36 PM, George Burgess IV via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: gbiv
> Date: Wed Jan 13 17:36:34 2016
> New Revision: 257710
>
> URL: http://llvm.org/viewvc/llvm-p
Author: hans
Date: Wed Jan 13 18:01:17 2016
New Revision: 257720
URL: http://llvm.org/viewvc/llvm-project?rev=257720&view=rev
Log:
Merging r257696:
r257696 | marshall | 2016-01-13 14:52:36 -0800 (Wed, 13 Jan 2016) | 1 line
F
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, 2016 at 3:36 PM, George Burgess IV via cfe-commits
> wrote:
>>
>> Author: gbiv
>> Dat
Author: hans
Date: Wed Jan 13 18:02:08 2016
New Revision: 257721
URL: http://llvm.org/viewvc/llvm-project?rev=257721&view=rev
Log:
Merging r257702:
r257702 | marshall | 2016-01-13 15:05:15 -0800 (Wed, 13 Jan 2016) | 1 line
B
Thanks! Merged as indicated below.
On Wed, Jan 13, 2016 at 3:48 PM, Marshall Clow wrote:
> r257702 | marshall | 2016-01-13 15:05:15 -0800 (Wed, 13 Jan 2016) | 1 line
>
> Better comments in test. NFC
r257721.
> r257696 | marshall | 2016-01-13 14:52:36 -0800 (Wed, 13 Jan 2016) | 1 line
>
> Fix te
nwilson created this revision.
nwilson added reviewers: rsmith, faisalv, hubert.reinterpretcast, aaron.ballman.
nwilson added a subscriber: cfe-commits.
Adding checks and diagnostics which fall under Concepts TS[dcl.spec.concept]p5:
function concepts are required to have 'bool' return type.
Addin
nwilson added a comment.
Please let me know if the subject or summary of this Patch is ambiguous.
http://reviews.llvm.org/D16163
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 123 matches
Mail list logo
