mgrang created this revision.
mgrang added a project: clang.
This fixes failure in Misc/diag-template-diffing.cpp uncovered by
https://reviews.llvm.org/D39245.
https://reviews.llvm.org/D39944
Files:
lib/Sema/SemaOverload.cpp
Index: lib/Sema/SemaOverload.cpp
Author: marshall
Date: Sun Nov 12 10:48:42 2017
New Revision: 317996
URL: http://llvm.org/viewvc/llvm-project?rev=317996&view=rev
Log:
Updated C++2a status page with new features/defects approved in Albuquerque.
Modified:
libcxx/trunk/www/cxx2a_status.html
Modified: libcxx/trunk/www/cxx2a_st
Author: marshall
Date: Sun Nov 12 10:52:16 2017
New Revision: 318000
URL: http://llvm.org/viewvc/llvm-project?rev=318000&view=rev
Log:
Two more papers from Albuquerque
Modified:
libcxx/trunk/www/cxx2a_status.html
Modified: libcxx/trunk/www/cxx2a_status.html
URL:
http://llvm.org/viewvc/llvm-
rjmccall added a comment.
In https://reviews.llvm.org/D39462#922844, @lebedev.ri wrote:
> In https://reviews.llvm.org/D39462#922826, @rjmccall wrote:
>
> > I don't speak for the entire project, but I'm not sure I'm interested in
> > the diagnostic you're actually offering to contribute here. It
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
Hmm. It looks like the intent is for CompareOverloadCandidatesForDisplay to be
a total order, but I'm sure there are cases where it isn't. Okay, LGTM.
https://reviews.llvm.org/D39944
Agreed! Just wanted to do it but then I noticed that there's already one
that landed in r307791! :)
On Sun, Oct 29, 2017 at 2:06 PM, Kim Gräsman wrote:
> A clang-tidy check to remove empty messages from source would be nice,
> though...
>
> - Kim
>
> Den 27 okt. 2017 10:24 fm skrev "Nicolas Less
rjmccall added a comment.
I don't speak for the entire project, but I'm not sure I'm interested in the
diagnostic you're actually offering to contribute here. It may produce a
warning on your specific test case, but I think it's really much too rigid and
will lead to massive false positives.
smeenai added a comment.
In https://reviews.llvm.org/D39462#922847, @rjmccall wrote:
> In https://reviews.llvm.org/D39462#922844, @lebedev.ri wrote:
>
> > In https://reviews.llvm.org/D39462#922826, @rjmccall wrote:
> >
> > > I don't speak for the entire project, but I'm not sure I'm interested in
lebedev.ri added a comment.
In https://reviews.llvm.org/D39462#922826, @rjmccall wrote:
> I don't speak for the entire project, but I'm not sure I'm interested in the
> diagnostic you're actually offering to contribute here. It may produce a
> warning on your specific test case, but I think it
mgrang created this revision.
mgrang added a project: clang.
This fixes the following failures uncovered by https://reviews.llvm.org/D39245:
Clang :: OpenMP/task_firstprivate_codegen.cpp
Clang :: OpenMP/task_private_codegen.cpp
Clang :: OpenMP/taskloop_firstprivate_codegen.cpp
Clang :: Op
mgrang added a comment.
Although this patches fixes the above unit test failures, the generated code is
very different from the one that the tests expect. As a result, these tests
need to be adjusted. Could the reviewers please comment/suggest on whether it
is ok to fix the tests as a result of
On Thu, Nov 09, 2017 at 05:37:32PM -0800, Volodymyr Sapsai wrote:
> On Nov 9, 2017, at 12:13, Reimar Döffinger wrote:
> >
> > Hello!
> >
> > On Wed, Nov 08, 2017 at 12:36:00PM -0800, Volodymyr Sapsai wrote:
> >> Thanks for the patch, Reimar. Can you please add tests to ensure this
> >> function
xgsa added a comment.
In https://reviews.llvm.org/D39622#919579, @aprantl wrote:
> For clarification: what is the "symbols table" you are referring to in the
> description?
I meant the data dumped with "nm -an ./test".
By the way, I haven't abandoned the patch, I have found one more case when
kastiglione created this revision.
Herald added a subscriber: klimek.
Allow the `isDefinition()` matcher to apply to `ObjCMethodDecl` nodes, in
addition to those it already supports. For whatever reason, `ObjCMethodDecl`
does not inherit from `FunctionDecl` and so this is specialization is necessa
rjmccall added a comment.
I see. The problem now, though, is that things involving, say, a size_t and a
numeric_limits will never warn.
Repository:
rL LLVM
https://reviews.llvm.org/D39462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
rjmccall added a comment.
In https://reviews.llvm.org/D39947#922870, @mgrang wrote:
> Although this patches fixes the above unit test failures, the generated code
> is very different from the one that the tests expect. As a result, these
> tests need to be adjusted. Could the reviewers please c
rwols updated this revision to Diff 122606.
rwols added a comment.
- Merge with upstream, fix merge conflicts
- Add a FIXME for a caching strategy for .clang-format files
https://reviews.llvm.org/D39430
Files:
clangd/ClangdLSPServer.cpp
clangd/ClangdServer.cpp
clangd/ClangdServer.h
clan
rwols marked 4 inline comments as done.
rwols added a comment.
Is this ready to merge? I'd like to implement tests in another differential,
I'm having trouble referencing a temp dir from `lit` in a JSON request to
clangd.
Comment at: clangd/ClangdServer.cpp:429
+ auto Tagged
lebedev.ri added a comment.
In https://reviews.llvm.org/D39462#922847, @rjmccall wrote:
> In https://reviews.llvm.org/D39462#922844, @lebedev.ri wrote:
>
> > In https://reviews.llvm.org/D39462#922826, @rjmccall wrote:
> >
> > > I don't speak for the entire project, but I'm not sure I'm interested
rjmccall added a comment.
That's the template type-propagation problem: the template was invoked with a
size_t, but that's erased down to the canonical type by the template machinery.
That can't be fixed within the template, but at use sites, we could
theoretically recognize that e.g. the resu
rjmccall added a comment.
I think the implementation looks fine. The metadata design also sounds good to
me, but I'd like Hal to weigh in before you commit.
Repository:
rL LLVM
https://reviews.llvm.org/D39455
___
cfe-commits mailing list
cfe-co
sgolemon added a comment.
Fixed in parallel by https://reviews.llvm.org/D39013
https://reviews.llvm.org/D38892
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mgrang added a comment.
In https://reviews.llvm.org/D39947#922889, @rjmccall wrote:
> In https://reviews.llvm.org/D39947#922870, @mgrang wrote:
>
> > Although this patches fixes the above unit test failures, the generated
> > code is very different from the one that the tests expect. As a result
Author: marshall
Date: Sun Nov 12 19:59:22 2017
New Revision: 318011
URL: http://llvm.org/viewvc/llvm-project?rev=318011&view=rev
Log:
Implement P0550R2: Transformation Trait remove_cvref
Added:
libcxx/trunk/test/std/utilities/meta/meta.trans/meta.trans.other/remove_cvref.pass.cpp
Modified:
Author: marshall
Date: Sun Nov 12 20:15:39 2017
New Revision: 318012
URL: http://llvm.org/viewvc/llvm-project?rev=318012&view=rev
Log:
Put the status in the wrong column
Modified:
libcxx/trunk/www/cxx2a_status.html
Modified: libcxx/trunk/www/cxx2a_status.html
URL:
http://llvm.org/viewvc/llv
jina.nahias added inline comments.
Comment at: test/CodeGen/avx512f-builtins.c:6231
+ // CHECK: bitcast <16 x i1> %{{.*}} to i16
+ // CHECK: and i32 %{{.*}}, 255
+ // CHECK: shl i32 %{{.*}}, 8
craig.topper wrote:
> Does this really produce kunpackb in the back
26 matches
Mail list logo
