rkirsling created this revision.
rkirsling added reviewers: djasper, klimek.
Herald added a subscriber: cfe-commits.
WebKit C++ style for object initialization is as follows:
Foo foo { bar };
Yet using `clang-format -style=webkit` changes this to:
Foo foo{ bar };
As there is no existing co
Author: d0k
Date: Tue Apr 24 10:57:53 2018
New Revision: 330746
URL: http://llvm.org/viewvc/llvm-project?rev=330746&view=rev
Log:
[clangd] Initialize candidate count in unit test
Found by msan.
Modified:
clang-tools-extra/trunk/unittests/clangd/FindSymbolsTests.cpp
Modified: clang-tools-ext
lebedev.ri added inline comments.
Comment at: include/__config:1016
+// because GCC does not silence them via (void) cast.
+#if __has_cpp_attribute(nodiscard) && _LIBCPP_STD_VER > 17
+# define _LIBCPP_NODISCARD [[nodiscard]]
mclow.lists wrote:
> `[[nodiscard]]`
mclow.lists added inline comments.
Comment at: include/__config:1016
+// because GCC does not silence them via (void) cast.
+#if __has_cpp_attribute(nodiscard) && _LIBCPP_STD_VER > 17
+# define _LIBCPP_NODISCARD [[nodiscard]]
lebedev.ri wrote:
> mclow.lists wrot
mclow.lists added a comment.
In https://reviews.llvm.org/D41316#1054968, @efriedma wrote:
> Yes, the standard says you're allowed to throw an exception from the
> random_device constructor, or use a PRNG with an arbitrary seed, or even just
> return zeros from operator(). But none of those beha
lebedev.ri added inline comments.
Comment at: include/__config:1016
+// because GCC does not silence them via (void) cast.
+#if __has_cpp_attribute(nodiscard) && _LIBCPP_STD_VER > 17
+# define _LIBCPP_NODISCARD [[nodiscard]]
mclow.lists wrote:
> lebedev.ri wrote
Author: tra
Date: Tue Apr 24 11:23:19 2018
New Revision: 330753
URL: http://llvm.org/viewvc/llvm-project?rev=330753&view=rev
Log:
[CUDA] Enable CUDA compilation with CUDA-9.2
Differential Revision: https://reviews.llvm.org/D45827
Modified:
cfe/trunk/include/clang/Basic/Cuda.h
cfe/trunk/l
This revision was automatically updated to reflect the committed changes.
Closed by commit rL330753: [CUDA] Enable CUDA compilation with CUDA-9.2
(authored by tra, committed by ).
Herald added subscribers: llvm-commits, bixia.
Changed prior to commit:
https://reviews.llvm.org/D45827?vs=143124&i
Author: d0k
Date: Tue Apr 24 11:40:44 2018
New Revision: 330754
URL: http://llvm.org/viewvc/llvm-project?rev=330754&view=rev
Log:
[clangd] Commit a heinous crime to make test hermetic and not depend on the
standard library being around.
Added:
clang-tools-extra/trunk/test/clangd/Inputs/
jbcoe added a comment.
I can merge this for you. Congratulations on the commit!
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D45891
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
bcraig added a comment.
For those that would prefer random device to not exist if it isn't
cryptographically secure, please write a wg21 paper. I will gladly review such
a paper, and if you need a presenter, then I will present it if I am attending.
I won't be at Rapperswil, but I will be at
yaxunl updated this revision to Diff 143792.
yaxunl added a comment.
Remove CodeGenModule argument from addPrefix* functions.
https://reviews.llvm.org/D44984
Files:
include/clang/Basic/IdentifierTable.h
include/clang/Basic/LangOptions.def
include/clang/Frontend/FrontendOptions.h
include
tra added inline comments.
Comment at: test/CodeGenCUDA/device-stub.cu:2-8
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -emit-llvm %s \
+// RUN: -fcuda-include-gpubinary %t -o - | FileCheck %s
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -emit-llvm %s \
+// RUN: -fcuda-includ
mstorsjo updated this revision to Diff 143794.
mstorsjo added a comment.
Added an isEmpty() method.
Btw, did you see https://bugs.llvm.org/show_bug.cgi?id=37196? That one feels
quite a bit more convolved so I don't really know (so far) how to approach
fixing that.
https://reviews.llvm.org/D45
vsapsai planned changes to this revision.
vsapsai added inline comments.
Comment at: clang/test/Headers/stdatomic.cpp:4
+
+#include
+
mclow.lists wrote:
> Is there a reason we want to test this twice - once in clang and once in
> libc++?
> We can use `expected-
xazax.hun created this revision.
xazax.hun added a reviewer: alexfh.
Herald added subscribers: dkrupp, rnkovacs, baloghadamsoftware, whisperity.
This approach will also introduce false negatives.
A better approach would be to check if the null statement is the result of
folding an `if constexpr`.
Author: aaronballman
Date: Tue Apr 24 12:21:04 2018
New Revision: 330759
URL: http://llvm.org/viewvc/llvm-project?rev=330759&view=rev
Log:
Improve -Warray-bounds to handle multiple array extents rather than only
handling the top-most array extent.
Patch by Bevin Hansson.
Modified:
cfe/trunk
aaron.ballman closed this revision.
aaron.ballman added a comment.
Committed in r330759, thank you for the patch!
https://reviews.llvm.org/D45865
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/
mikhail.ramalho added a subscriber: arphaman.
mikhail.ramalho added a comment.
> Why wasn't there a file for function parameter? Function parameters *are*
> declared in some file, or am I missing something?
They are declared in some file defined by the line markers; the file are
not registered i
khuttun added a comment.
Thank you!
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D45891
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vsapsai updated this revision to Diff 143797.
vsapsai added a comment.
- Move tests to test/std.
https://reviews.llvm.org/D34331
Files:
libcxx/include/__functional_03
libcxx/include/functional
libcxx/test/std/utilities/function.objects/func.wrap/func.wrap.func/func.wrap.func.con/move_ree
yaxunl marked 2 inline comments as done.
yaxunl added inline comments.
Comment at: test/CodeGenCUDA/device-stub.cu:2-8
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -emit-llvm %s \
+// RUN: -fcuda-include-gpubinary %t -o - | FileCheck %s
+// RUN: %clang_cc1 -triple x86_64-linux-
NoQ added a comment.
Aha, ok, yeah, that sounds like a lot, thank you. I think i'll follow up with a
separate commit that will enable first-level-code-file-include analysis in all
files under an on-by-default `-analyzer-config` flag, would that make sense?
https://reviews.llvm.org/D45839
__
joerg added a comment.
I'm back to the point where I can't reproduce the problem :( Can we start
providing an actual failing test case? It's annoying to debug a problem when
you can't reproduce it.
https://reviews.llvm.org/D38680
___
cfe-commits m
yaxunl updated this revision to Diff 143800.
yaxunl marked 5 inline comments as done.
yaxunl added a comment.
Add back HIP related changes to the tests.
https://reviews.llvm.org/D44984
Files:
include/clang/Basic/IdentifierTable.h
include/clang/Basic/LangOptions.def
include/clang/Frontend/
mstorsjo added a comment.
In https://reviews.llvm.org/D38680#1077262, @joerg wrote:
> I'm back to the point where I can't reproduce the problem :( Can we start
> providing an actual failing test case? It's annoying to debug a problem when
> you can't reproduce it.
My testcase that triggers th
On 20 Sep 2017, at 20:06, Andrew Kaylor via cfe-commits
wrote:
>
> Author: akaylor
> Date: Wed Sep 20 11:06:44 2017
> New Revision: 313784
>
> URL: http://llvm.org/viewvc/llvm-project?rev=313784&view=rev
> Log:
> Remove offset size check in nullptr arithmetic handling
>
> Differential Revision
Author: dergachev
Date: Tue Apr 24 13:45:48 2018
New Revision: 330766
URL: http://llvm.org/viewvc/llvm-project?rev=330766&view=rev
Log:
[analyzer] Add support for the note diagnostic pieces to plist output format.
Note diagnostic pieces are an additional way of highlighting code sections to
the u
This revision was automatically updated to reflect the committed changes.
Closed by commit rC330766: [analyzer] Add support for the note diagnostic
pieces to plist output format. (authored by dergachev, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D45407
Files:
lib/StaticAn
This revision was automatically updated to reflect the committed changes.
Closed by commit rL330766: [analyzer] Add support for the note diagnostic
pieces to plist output format. (authored by dergachev, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://re
NoQ added a comment.
Yup thanks!~
Repository:
rC Clang
https://reviews.llvm.org/D45407
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zinovy.nis added inline comments.
Comment at: clang-tidy/modernize/RawStringLiteralCheck.cpp:71
+ // supported by specific code pages only.
+ if (Bytes.find_if_not(isASCII) != StringRef::npos)
+return false;
aaron.ballman wrote:
> zinovy.nis wrote:
> > aaro
Author: danalbert
Date: Tue Apr 24 14:18:37 2018
New Revision: 330770
URL: http://llvm.org/viewvc/llvm-project?rev=330770&view=rev
Log:
[Driver] Android triples are not aliases for other triples.
Summary:
Android targets should never use tools/libraries for non-Android
targets or vice versa.
Rev
This revision was automatically updated to reflect the committed changes.
Closed by commit rC330770: [Driver] Android triples are not aliases for other
triples. (authored by danalbert, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D45597?vs=142281&id=143807#toc
Repository:
Author: jbcoe
Date: Tue Apr 24 14:25:16 2018
New Revision: 330772
URL: http://llvm.org/viewvc/llvm-project?rev=330772&view=rev
Log:
[clang-tidy] Improve bugprone-unused-return-value check
Summary:
Add support for checking class template member functions.
Also add the following functions to be ch
This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE330772: [clang-tidy] Improve bugprone-unused-return-value
check (authored by jbcoe, committed by ).
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D45891
Files:
clang-tidy/bugprone/Un
vsapsai updated this revision to Diff 143817.
vsapsai added a comment.
- Tighten up a test with expected-error per review comment.
https://reviews.llvm.org/D45470
Files:
clang/lib/Headers/stdatomic.h
clang/test/Headers/stdatomic.cpp
libcxx/include/atomic
libcxx/test/libcxx/atomics/c_com
Author: danalbert
Date: Tue Apr 24 15:06:40 2018
New Revision: 330780
URL: http://llvm.org/viewvc/llvm-project?rev=330780&view=rev
Log:
Revert "[Driver] Android triples are not aliases for other triples."
Revering while I diagnose the failures.
This reverts commit 82dc3bf2157da280420f80e654292cb
efriedma created this revision.
efriedma added reviewers: fhahn, SjoerdMeijer.
Herald added a reviewer: javed.absar.
Herald added a subscriber: kristof.beyls.
Passing the features in random order will lead to unpredictable results when
some of the features are related (like the architecture-versi
efriedma updated this revision to Diff 143821.
efriedma added a comment.
Add REQUIRES line to testcase.
Repository:
rC Clang
https://reviews.llvm.org/D46030
Files:
lib/Basic/Targets.cpp
test/CodeGen/arm-build-attributes.c
Index: test/CodeGen/arm-build-attributes.c
=
AlexeySotkin added a comment.
It is not clear why we need two versions of get_pipe_num_packets and
get_pipe_max_packets builtins. There is only one instruction per builtin in the
SPIR-V spec. I think splitting the IR type is enough for translation to SPIR-V
purposes.
Repository:
rC Clang
h
AlexeySotkin added inline comments.
Comment at: lib/CodeGen/CGOpenCLRuntime.h:65
virtual llvm::Type *getPipeType(const PipeType *T);
+ virtual llvm::Type *getPipeType(const PipeType *T, StringRef Name,
+ llvm::Type *&PipeTy);
Charusso updated this revision to Diff 143825.
Charusso marked an inline comment as done.
Charusso added a comment.
Defined an option to switch off the `_s` suffixed functions usage.
https://reviews.llvm.org/D45050
Files:
clang-tidy/bugprone/BugproneTidyModule.cpp
clang-tidy/bugprone/CMakeL
paulsemel created this revision.
paulsemel added a reviewer: aaron.ballman.
This patch adds a test for long double format.
Repository:
rC Clang
https://reviews.llvm.org/D46033
Files:
test/CodeGen/dump-struct-builtin.c
Index: test/CodeGen/dump-struct-builtin.c
sammccall updated this revision to Diff 143836.
sammccall added a comment.
clang-format
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D46035
Files:
clangd/ClangdServer.cpp
clangd/ClangdUnit.cpp
clangd/ClangdUnit.h
clangd/CodeComplete.cpp
clangd/Protocol.h
clangd/Sou
rjmccall accepted this revision.
rjmccall added a comment.
Thank you.
https://reviews.llvm.org/D44984
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sammccall created this revision.
sammccall added a reviewer: hokein.
Herald added subscribers: cfe-commits, jkorous, MaskRay, ioeric, ilya-biryukov,
klimek.
The Language Server Protocol unfortunately mandates that locations in files
be represented by line/column pairs, where the "column" is actua
Charusso added a comment.
In https://reviews.llvm.org/D45050#1071897, @xbolva00 wrote:
> Shouldn't it catch in curl also this code?
>
> urllen = strlen(url_clone);
>
>
>
> memcpy(newest, url_clone, urllen);
>
> Edit: if possible, report these bugs to project developers :)
Thanks for your
sammccall updated this revision to Diff 143838.
sammccall added a comment.
Remove some debugging junk, tweak a comment.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D46035
Files:
clangd/ClangdServer.cpp
clangd/ClangdUnit.cpp
clangd/ClangdUnit.h
clangd/CodeComplete.cpp
sammccall added a subscriber: bkramer.
sammccall added a comment.
In https://reviews.llvm.org/D44882#1076927, @malaperle wrote:
> In https://reviews.llvm.org/D44882#1076864, @sammccall wrote:
>
> > So this fails if there's no standard library available without flags, which
> > is the case in goo
Author: rtrieu
Date: Tue Apr 24 17:31:15 2018
New Revision: 330789
URL: http://llvm.org/viewvc/llvm-project?rev=330789&view=rev
Log:
[ODRHash] Hash template arguments of methods.
Modified:
cfe/trunk/include/clang/Basic/DiagnosticSerializationKinds.td
cfe/trunk/lib/AST/ODRHash.cpp
cfe/
Author: yaxunl
Date: Tue Apr 24 18:10:37 2018
New Revision: 330790
URL: http://llvm.org/viewvc/llvm-project?rev=330790&view=rev
Log:
[HIP] Add hip input kind and codegen for kernel launching
HIP is a language similar to CUDA
(https://github.com/ROCm-Developer-Tools/HIP/blob/master/docs/markdown/
This revision was automatically updated to reflect the committed changes.
Closed by commit rL330790: [HIP] Add hip input kind and codegen for kernel
launching (authored by yaxunl, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D44984?v
NoQ created this revision.
NoQ added reviewers: dcoughlin, xazax.hun, a.sidorin, george.karpenkov, szepet.
Herald added subscribers: cfe-commits, rnkovacs, baloghadamsoftware.
This hotfix is similar to https://reviews.llvm.org/D43689 (and needs a
follow-up similar to https://reviews.llvm.org/D442
NoQ updated this revision to Diff 143850.
NoQ added a comment.
Add a test similar to pr37166. It should be actually fixed now with this patch.
Essentially, we may also be lifetime-extended by an array of aggregate
structures (with arbitrary amounts of interleaving array and aggregate
structure
rjmccall added a comment.
I agree that this is one of those warnings that just can't be done in any
reasonable way.
https://reviews.llvm.org/D45601
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listin
malaperle added a comment.
In https://reviews.llvm.org/D44882#1077516, @sammccall wrote:
> It makes sense, but @bkramer came up with some deep magic in
> https://reviews.llvm.org/rL330754 so I think we're actually good now.
Nice! Thanks @bkramer !
Repository:
rCTE Clang Tools Extra
https:
Author: bion
Date: Tue Apr 24 18:58:55 2018
New Revision: 330791
URL: http://llvm.org/viewvc/llvm-project?rev=330791&view=rev
Log:
[libcxx] [test] Remove nonportable that errc::is_a_directory produces "Is a
directory" from ios_base::failure tests
These io_error asserts that std::errc::is_a_direc
BillyONeal closed this revision.
BillyONeal added a comment.
D:\msvc\src\qa\VC\Libs\libcxx\upstream>git svn dcommit
Committing to https://llvm.org/svn/llvm-project/libcxx/trunk ...
Authentication realm: https://llvm.org:443 LLVM Subversion repository
Password for 'bion':
M
test/std/input
BillyONeal added a comment.
@mclow.lists Are you OK with this one? It's conceptually similar to the
previous review.
https://reviews.llvm.org/D45294
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
Author: yaxunl
Date: Tue Apr 24 19:34:04 2018
New Revision: 330793
URL: http://llvm.org/viewvc/llvm-project?rev=330793&view=rev
Log:
Fix failure in lit test kernel-call.cu
There is signext on ppc64. Just remove check for function argument.
Modified:
cfe/trunk/test/CodeGenCUDA/kernel-call.cu
Author: faisalv
Date: Tue Apr 24 19:42:26 2018
New Revision: 330794
URL: http://llvm.org/viewvc/llvm-project?rev=330794&view=rev
Log:
[c++2a] [concepts] Add rudimentary parsing support for template concept
declarations
This patch is a tweak of changyu's patch: https://reviews.llvm.org/D40381. I
Author: yaxunl
Date: Tue Apr 24 20:16:07 2018
New Revision: 330795
URL: http://llvm.org/viewvc/llvm-project?rev=330795&view=rev
Log:
Fix lit test kernel-call.cu failure on ps4 due to dso_local
Modified:
cfe/trunk/test/CodeGenCUDA/kernel-call.cu
Modified: cfe/trunk/test/CodeGenCUDA/kernel-cal
Author: faisalv
Date: Tue Apr 24 20:28:23 2018
New Revision: 330796
URL: http://llvm.org/viewvc/llvm-project?rev=330796&view=rev
Log:
Fix rC330794 - a parameter that should have been dependent was inadvertently
not -
and compiled in MSVC - but not so for the other bots.
The fix was to make it d
Author: faisalv
Date: Tue Apr 24 20:54:20 2018
New Revision: 330798
URL: http://llvm.org/viewvc/llvm-project?rev=330798&view=rev
Log:
[NFC] Make dependent parameter non-deducible, so that we are forced to use the
default template parameter.
This might provide users with more graceful diagnostics
rjmccall created this revision.
Herald added a reviewer: javed.absar.
Herald added subscribers: cfe-commits, kristof.beyls.
This fixes two major problems:
- We were not capping vector alignment as desired on 32-bit ARM.
- We were using different alignments based on the AVX settings on Intel, so w
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
Fix seems right.
Repository:
rC Clang
https://reviews.llvm.org/D45755
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists
>
> Why wasn't there a file for function parameter? Function parameters *are*
> declared in some file, or am I missing something?
>
>
They are declared in some file defined by the line markers; the file are
not registered in the SourceManager as actual files, so getting the
FileEntry will always fa
101 - 168 of 168 matches
Mail list logo
