Author: akirtzidis
Date: Sat Aug 25 23:27:23 2018
New Revision: 340696
URL: http://llvm.org/viewvc/llvm-project?rev=340696&view=rev
Log:
[index] Introduce 'ProtocolInterface' as part of SymbolPropertySet
This is useful to directly infer that a method or property is from a protocol
interface
at t
lichray created this revision.
lichray added reviewers: mclow.lists, ldionne.
Herald added a reviewer: EricWF.
Herald added a subscriber: christof.
Avoid buffer overflow by replacing the pointer interface with an array
reference interface in C++2a.
Tentatively ready on Batavia2018.
https://wg21.
andrewrk updated this revision to Diff 162574.
andrewrk added a comment.
Addressed review feedback
Repository:
rC Clang
https://reviews.llvm.org/D51265
Files:
lib/Headers/stdarg.h
lib/Headers/stddef.h
Index: lib/Headers/stddef.h
=
Qix- added inline comments.
Comment at: lib/Headers/stdarg.h:30
#ifndef _VA_LIST
+#ifndef _VA_LIST_T
typedef __builtin_va_list va_list;
Super nit-picky but you could condense this a bit by using
```
#if !defined(_VA_LIST) && !defined(_VA_LIST_T)
```
and a sin
andrewrk created this revision.
andrewrk added reviewers: eli.friedman, krememek, ddunbar, lattner.
andrewrk added a project: clang.
Herald added a subscriber: cfe-commits.
stdarg.h:
- va_list - macos _va_list.h was duplicately defining va_list. Fixed by this
ifndef _VA_LIST_T
- additionally def
stringham added a comment.
@djasper can you give some direction here?
Would you be okay with me extending the BracketAlignmentStyle option?
What do I need to do to get this change merged?
https://reviews.llvm.org/D33029
___
cfe-commits mailing list
owenpan added a comment.
@stringham Here is what I did to run the unit tests:
cd build
make FormatTests
tools/clang/unittests/Format/FormatTests
https://reviews.llvm.org/D33029
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://li
hfinkel added a comment.
In https://reviews.llvm.org/D37624#1211748, @apazos wrote:
> Hello folks, is there a plan to merge this feature still?
There's still a desire for the underlying functionality. We still need, as I
recall, to figure out what's supposed to happen for C++ functions.
http
aaronpuchert updated this revision to Diff 162562.
aaronpuchert added a comment.
Use Locked flag to determine whether to unlock on destruction
Instead of unlocking the mutexes that are still available while not complaining
about those that aren't, we use the status of the scoped capability to
d
mclow.lists added a comment.
I should also mention that as a conforming extension, I have implemented the
non-numeric bit operations for `std::byte`
https://reviews.llvm.org/D51262
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://list
mclow.lists created this revision.
mclow.lists added reviewers: EricWF, ldionne.
Herald added a subscriber: christof.
LWG adopted https://wg21.link/P0553 in Rapperswil, and suggested minor changes
to https://wg21.link/P0556.
They kind of go together; for example, `ispow2` is easily implemented u
bobsayshilol added a comment.
Pinging this as it's been "Ready to Land" for a while and I want to check that
that's not because I've forgotten to do something?
Repository:
rCXX libc++
https://reviews.llvm.org/D50101
___
cfe-commits mailing list
Wawha added a comment.
Hi klimek,
do you have time to take a look again to this patch?
Is my last patch ok for you?
best regards,
François
Repository:
rC Clang
https://reviews.llvm.org/D44609
___
cfe-commits mailing list
cfe-commits@lists.llv
zroug added a comment.
I'd also like to note that this is the code style preferred by most modern code
formatters that I know of and use:
- rustfmt for rust code
- prettier for javascript code
- black for python code
https://reviews.llvm.org/D33029
__
steveire created this revision.
Herald added a subscriber: cfe-commits.
This allows loading a file with pre-defined let commands for example.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D51261
Files:
clang-query/tool/ClangQuery.cpp
Index: clang-query/tool/ClangQuery.cpp
=
steveire created this revision.
Herald added a subscriber: cfe-commits.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D51260
Files:
clang-query/tool/ClangQuery.cpp
Index: clang-query/tool/ClangQuery.cpp
===
--- c
steveire created this revision.
Herald added a subscriber: cfe-commits.
Repository:
rC Clang
https://reviews.llvm.org/D51258
Files:
include/clang/ASTMatchers/Dynamic/Parser.h
lib/ASTMatchers/Dynamic/Parser.cpp
Index: lib/ASTMatchers/Dynamic/Parser.cpp
=
steveire created this revision.
Herald added a subscriber: cfe-commits.
Repository:
rC Clang
https://reviews.llvm.org/D51259
Files:
lib/ASTMatchers/Dynamic/Parser.cpp
unittests/ASTMatchers/Dynamic/ParserTest.cpp
Index: unittests/ASTMatchers/Dynamic/ParserTest.cpp
This revision was automatically updated to reflect the committed changes.
Closed by commit rL340681: [CUDA/OpenMP] Define only some host macros during
device compilation (authored by Hahnfeld, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm
Author: hahnfeld
Date: Sat Aug 25 06:42:40 2018
New Revision: 340681
URL: http://llvm.org/viewvc/llvm-project?rev=340681&view=rev
Log:
[CUDA/OpenMP] Define only some host macros during device compilation
When compiling CUDA or OpenMP device code Clang parses header files
that expect certain prede
Hahnfeld added a comment.
In https://reviews.llvm.org/D50845#1212643, @tra wrote:
> Please keep an eye on CUDA buildbot
> http://lab.llvm.org:8011/builders/clang-cuda-build.
> It runs fair amount of tests with libc++ and handful of libstdc++ versions
> and may a canary if these changes break s
Hahnfeld updated this revision to Diff 162543.
Hahnfeld added a comment.
Based on libc++ I guessed some more macros that may be needed on macOS and
Windows. As I can't test myself if somebody else could report if this change is
regressing CUDA support on these platforms.
https://reviews.llvm.o
22 matches
Mail list logo
