calixte created this revision.
calixte added reviewers: marco-c, vsk.
Herald added a subscriber: cfe-commits.
These options are taking regex separated by colons to filter files.
- if both are empty then all files are instrumented
- if -coverage-filter is empty then all the filenames matching any
kbobyrev accepted this revision.
kbobyrev added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D51989#1232099, @ilya-biryukov wrote:
> PS it'd be cool to have an interface to cl that does not rely on global
> state...
Yeah, I'm not sure if that's trivia
ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
LGTM
Comment at: clang-tools-extra/clangd/benchmarks/IndexBenchmark.cpp:52
+ if (!JSONArray->getAsArray()) {
+llvm::errs() << "Error when parsing JSON arra
sammccall added a comment.
Sorry for all the back and forth on this patch, but I have to ask... what's up
with switching to lit tests?
We've mostly started avoiding these as they're hard to maintain and debug (not
to mention write... crazy sed tricks!)
They're mostly used in places where we ne
kbobyrev updated this revision to Diff 165280.
kbobyrev marked 2 inline comments as done.
https://reviews.llvm.org/D51971
Files:
clang-tools-extra/clangd/benchmarks/IndexBenchmark.cpp
clang-tools-extra/test/clangd/Inputs/requests.json
clang-tools-extra/test/clangd/Inputs/requests.log
clan
Author: omtcyfz
Date: Thu Sep 13 07:21:50 2018
New Revision: 342137
URL: http://llvm.org/viewvc/llvm-project?rev=342137&view=rev
Log:
[clangd] Use JSON format in benchmark requests reader
After `FuzzyFindRequest` JSON (de)serialization was introduced, it
should replace ad-hoc fuzzy-find request p
This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE342137: [clangd] Use JSON format in benchmark requests
reader (authored by omtcyfz, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D51971?vs=165280&id=165281#toc
Repository:
rCTE
kadircet added a comment.
In https://reviews.llvm.org/D51747#1233371, @sammccall wrote:
> Sorry for all the back and forth on this patch, but I have to ask... what's
> up with switching to lit tests?
>
> We've mostly started avoiding these as they're hard to maintain and debug
> (not to mention
kbobyrev updated this revision to Diff 165283.
kbobyrev added a comment.
Update `requests.json` to comply with the new format.
https://reviews.llvm.org/D52028
Files:
clang-tools-extra/clangd/CodeComplete.cpp
clang-tools-extra/clangd/FindSymbols.cpp
clang-tools-extra/clangd/index/Index.cpp
Author: omtcyfz
Date: Thu Sep 13 07:27:03 2018
New Revision: 342138
URL: http://llvm.org/viewvc/llvm-project?rev=342138&view=rev
Log:
[clangd] Cleanup FuzzyFindRequest filtering limit semantics
As discussed during D51860 review, it is better to use `llvm::Optional`
here as it has clear semantics
This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE342138: [clangd] Cleanup FuzzyFindRequest filtering limit
semantics (authored by omtcyfz, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D52028?vs=165283&id=165284#toc
Repository:
Author: xbolva00
Date: Thu Sep 13 07:27:32 2018
New Revision: 342139
URL: http://llvm.org/viewvc/llvm-project?rev=342139&view=rev
Log:
Print correctly dependency paths on Windows
Summary:
Before:
main.o: main.c ../include/lib\test.h
After:
main.o: main.c ../include/lib/test.h
Fixes PR38877
Rev
This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rC342139: Print correctly dependency paths on Windows
(authored by xbolva00, committed by ).
Repository:
rC Clang
https:
kbobyrev added a subscriber: rnk.
kbobyrev added a comment.
In https://reviews.llvm.org/D52030#1233349, @sammccall wrote:
> In https://reviews.llvm.org/D52030#1233323, @kbobyrev wrote:
>
> > Looks good, thanks!
> >
> > While we're here: I'm wondering whether we should also introduce very basic
>
kbobyrev added a comment.
In https://reviews.llvm.org/D52030#1233395, @kbobyrev wrote:
> In https://reviews.llvm.org/D52030#1233349, @sammccall wrote:
>
> > In https://reviews.llvm.org/D52030#1233323, @kbobyrev wrote:
> >
> > > Looks good, thanks!
> > >
> > > While we're here: I'm wondering wheth
sammccall added a comment.
In https://reviews.llvm.org/D51747#1233377, @kadircet wrote:
> In https://reviews.llvm.org/D51747#1233371, @sammccall wrote:
>
> > Sorry for all the back and forth on this patch, but I have to ask... what's
> > up with switching to lit tests?
> >
> > We've mostly start
miyuki created this revision.
miyuki added reviewers: NoQ, dcoughlin, george.karpenkov.
Herald added subscribers: Szelethus, mikhail.ramalho, a.sidorin, szepet,
eraman, xazax.hun.
A recent change https://reviews.llvm.org/D50545 started using diff to
compare plist output against reference output.
kbobyrev updated this revision to Diff 165297.
kbobyrev added a comment.
Don't create abstractions for now.
https://reviews.llvm.org/D51982
Files:
clang-tools-extra/clangd/CMakeLists.txt
clang-tools-extra/clangd/index/dex/Dex.cpp
clang-tools-extra/clangd/index/dex/Dex.h
clang-tools-extr
kbobyrev updated this revision to Diff 165298.
kbobyrev added a comment.
Remove artifacts from assertion messages.
https://reviews.llvm.org/D51982
Files:
clang-tools-extra/clangd/CMakeLists.txt
clang-tools-extra/clangd/index/dex/Dex.cpp
clang-tools-extra/clangd/index/dex/Dex.h
clang-too
kadircet updated this revision to Diff 165302.
kadircet added a comment.
This revision is now accepted and ready to land.
- Turn back to unit tests.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D51747
Files:
clangd/ClangdServer.cpp
unittests/clangd/ClangdTests.cpp
Index:
kadircet added a comment.
In https://reviews.llvm.org/D51747#1233401, @sammccall wrote:
> In https://reviews.llvm.org/D51747#1233377, @kadircet wrote:
>
> > In https://reviews.llvm.org/D51747#1233371, @sammccall wrote:
> >
> > > Sorry for all the back and forth on this patch, but I have to ask...
Author: omtcyfz
Date: Thu Sep 13 08:35:55 2018
New Revision: 342143
URL: http://llvm.org/viewvc/llvm-project?rev=342143&view=rev
Log:
[clangd] Fix Dexp build
%s/MaxCandidateCount/Limit/g after rL342138.
Modified:
clang-tools-extra/trunk/clangd/index/dex/dexp/Dexp.cpp
Modified: clang-tools-e
christylee added inline comments.
Comment at: lib/Lex/PPDirectives.cpp:1901
+FilenameLoc,
+LangOpts.MSVCCompat ? NormalizedPath.c_str() : Filename, false,
+LookupFrom, LookupFromFile, CurDir,
rsmith wrote:
> You're passing in `
ilya-biryukov added a comment.
In https://reviews.llvm.org/D51747#1233499, @kadircet wrote:
> Np, not planning to land it before we figure out the issue with vim can't
> handling lots of diagnostics anyway.
Just wondering: what's the problem with Vim? Slowness? Bad UI for diagnostics
when the
christylee updated this revision to Diff 165311.
christylee added a comment.
Addressed @rsmith 's comments.
https://reviews.llvm.org/D51333
Files:
include/clang/Basic/DiagnosticLexKinds.td
lib/Lex/PPDirectives.cpp
Index: lib/Lex/PPDirectives.cpp
===
sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.
Great!
Comment at: clang-tools-extra/clangd/index/dex/Iterator.cpp:11
#include "Iterator.h"
+#include "PostingList.h"
#include
And here
==
kbobyrev updated this revision to Diff 165318.
kbobyrev marked an inline comment as done.
https://reviews.llvm.org/D51982
Files:
clang-tools-extra/clangd/CMakeLists.txt
clang-tools-extra/clangd/index/dex/Dex.cpp
clang-tools-extra/clangd/index/dex/Dex.h
clang-tools-extra/clangd/index/dex/I
kbobyrev updated this revision to Diff 165320.
kbobyrev added a comment.
Wording: traversed *in order*.
https://reviews.llvm.org/D51982
Files:
clang-tools-extra/clangd/CMakeLists.txt
clang-tools-extra/clangd/index/dex/Dex.cpp
clang-tools-extra/clangd/index/dex/Dex.h
clang-tools-extra/cl
kbobyrev marked an inline comment as done.
kbobyrev added inline comments.
Comment at: clang-tools-extra/clangd/index/dex/Iterator.h:39
#include
+#include "PostingList.h"
sammccall wrote:
> Why this dep? Seems circular
`Iterator` interface uses `DocID`, so I
kbobyrev added inline comments.
Comment at: clang-tools-extra/clangd/index/dex/Iterator.h:39
#include
+#include "PostingList.h"
kbobyrev wrote:
> sammccall wrote:
> > Why this dep? Seems circular
> `Iterator` interface uses `DocID`, so I guess it should depen
Author: abataev
Date: Thu Sep 13 09:54:05 2018
New Revision: 342151
URL: http://llvm.org/viewvc/llvm-project?rev=342151&view=rev
Log:
[OPENMP] Fix PR38903: Crash on instantiation of the non-dependent
declare reduction.
If the declare reduction construct with the non-dependent type is
defined in t
Author: erichkeane
Date: Thu Sep 13 09:58:24 2018
New Revision: 342152
URL: http://llvm.org/viewvc/llvm-project?rev=342152&view=rev
Log:
[NFC]Refactor MultiVersion Resolver Emission to combine types
Previously, both types (plus the future target-clones) of
multiversioning had a separate ResolverO
JonasToth updated this revision to Diff 165321.
JonasToth added a comment.
This is the newer and better working code to slice and separate multiple
vardecls in a multi-decl stmt.
I want this version only for now, as the multiple variable definitions in one
statement are by far the most common vari
tra added a comment.
@rsmith ping.
https://reviews.llvm.org/D51808
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kbobyrev updated this revision to Diff 165324.
kbobyrev marked 3 inline comments as done.
kbobyrev added a comment.
Pull `DocID` to `Iterator.h`.
https://reviews.llvm.org/D51982
Files:
clang-tools-extra/clangd/CMakeLists.txt
clang-tools-extra/clangd/index/dex/Dex.cpp
clang-tools-extra/cla
Author: omtcyfz
Date: Thu Sep 13 10:11:03 2018
New Revision: 342155
URL: http://llvm.org/viewvc/llvm-project?rev=342155&view=rev
Log:
[clangd] Introduce PostingList interface
This patch abstracts `PostingList` interface and reuses existing
implementation. It will be used later to test different `
This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE342155: [clangd] Introduce PostingList interface (authored
by omtcyfz, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D51982?vs=165324&id=165325#toc
Repository:
rCTE Clang Tools
kbobyrev created this revision.
kbobyrev added reviewers: ioeric, ilya-biryukov, sammccall.
kbobyrev added a project: clang-tools-extra.
Herald added subscribers: kadircet, arphaman, jkorous, MaskRay.
While this looks like misuse of benchmark library, this can be still when
experimenting with ind
shuaiwang added a comment.
Just some quick comments, I'll take a deeper look into other comments later.
This diff along unfortunately won't be able to handle `emplace_back` just yet,
the reason (I believe, haven't fully tested) is that `std::forward` is not
handled properly and almost all std f
JonasToth added inline comments.
Comment at: clang-tidy/readability/IsolateDeclCheck.cpp:35
+namespace {
+SourceLocation findNextTerminator(SourceLocation Start, const SourceManager
&SM,
+ const LangOptions &LangOpts) {
Remove du
ldionne marked 2 inline comments as done.
ldionne added inline comments.
Comment at: clang/lib/Sema/SemaTemplateInstantiateDecl.cpp:1268
+ // Instantiate the attributes on both the template declaration and the
associated record declaration.
+ SemaRef.InstantiateAttrsForDecl(Te
JonasToth added a comment.
In https://reviews.llvm.org/D52008#1233667, @shuaiwang wrote:
> Just some quick comments, I'll take a deeper look into other comments later.
>
> This diff along unfortunately won't be able to handle `emplace_back` just
> yet, the reason (I believe, haven't fully tested
lebedev.ri added a comment.
In https://reviews.llvm.org/D52008#1233667, @shuaiwang wrote:
> Just some quick comments, I'll take a deeper look into other comments later.
>
> This diff along unfortunately won't be able to handle `emplace_back` just yet
My apologies, for some reason i though it wa
glaubitz added a comment.
I have just rebased the patch and I am looking at the tests now.
Repository:
rC Clang
https://reviews.llvm.org/D43630
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
rsmith added inline comments.
Comment at: clang/lib/Sema/SemaTemplateInstantiateDecl.cpp:1268
+ // Instantiate the attributes on both the template declaration and the
associated record declaration.
+ SemaRef.InstantiateAttrsForDecl(TemplateArgs, Pattern, Inst, LateAttrs,
Star
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Looks good (with a couple of minor code style adjustments). Do you need someone
to commit this for you?
In https://reviews.llvm.org/D51333#1233042, @sammccall wrote:
> One thing I'd like to d
glaubitz added a comment.
I don't think that "test/Driver/linux-header-search.cpp" needs to be updated.
x32 is actually supported in both an x86_64-linux-gnu environment and
x86_64-linux-gnux32 environment. In order to update
test/Driver/linux-header-search.cpp, we would have to add a debian-10
glaubitz added a comment.
I have pushed a rebase revision here: https://reviews.llvm.org/D52050
I have no idea how to edit the author though.
Repository:
rC Clang
https://reviews.llvm.org/D43630
___
cfe-commits mailing list
cfe-commits@lists.llv
glaubitz created this revision.
glaubitz added reviewers: rsmith, dschuff.
Herald added a subscriber: cfe-commits.
When targeting x32, the x32 libraries and headers should be used, not
the x86_64 ones (which may not even be available), so prioritise those
and use the right multiarch triple.
Repo
xbolva00 added a comment.
Where here, I would love to have a fixint hints for unknown functions.
E.g.
std::forward somewhere in code-> fixit: "try to add #include "
https://reviews.llvm.org/D51333
___
cfe-commits mailing list
cfe-commits@lists.llvm
christylee updated this revision to Diff 165349.
https://reviews.llvm.org/D51333
Files:
include/clang/Basic/DiagnosticLexKinds.td
lib/Lex/PPDirectives.cpp
Index: lib/Lex/PPDirectives.cpp
===
--- lib/Lex/PPDirectives.cpp
+++ lib
christylee updated this revision to Diff 165350.
https://reviews.llvm.org/D51333
Files:
include/clang/Basic/DiagnosticLexKinds.td
lib/Lex/PPDirectives.cpp
Index: lib/Lex/PPDirectives.cpp
===
--- lib/Lex/PPDirectives.cpp
+++ lib
christylee added a comment.
@rsmith , thanks for the review, I fixed the variable capitalization. If you
could land it for me that'll be awesome!
In https://reviews.llvm.org/D51333#1233042, @sammccall wrote:
> One thing I'd like to do sometime is add code completion of filenames in
> #include
lebedev.ri added a comment.
The tests seem to have disappeared form the diff.
https://reviews.llvm.org/D51333
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vsk added a comment.
Please document the filter behavior (see docs/UsersManual.rst).
Comment at: include/clang/Driver/CC1Options.td:236
+def coverage_exclude_EQ : Joined<["-"], "coverage-exclude=">,
+ Alias;
def coverage_exit_block_before_body : Flag<["-"],
"coverage-exit-bl
sammccall accepted this revision.
sammccall added a comment.
Awesome, thank you.
Just a couple of last nits.
Comment at: unittests/clangd/ClangdTests.cpp:984
+std::vector Diagnostics) override {
+ std::lock_guard Lock(Mutex);
+ for(const Di
Author: srhines
Date: Thu Sep 13 12:50:02 2018
New Revision: 342165
URL: http://llvm.org/viewvc/llvm-project?rev=342165&view=rev
Log:
Support -fno-omit-frame-pointer with -pg.
Summary:
Previously, any instance of -fomit-frame-pointer would make it such that
-pg was an invalid flag combination. If
This revision was automatically updated to reflect the committed changes.
Closed by commit rL342165: Support -fno-omit-frame-pointer with -pg. (authored
by srhines, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D51713
Files:
cfe/trunk/
christylee updated this revision to Diff 165361.
christylee added a comment.
Added tests.
https://reviews.llvm.org/D51333
Files:
include/clang/Basic/DiagnosticLexKinds.td
lib/Lex/PPDirectives.cpp
test/Preprocessor/empty_file_to_include.h
test/Preprocessor/include-likely-typo.c
Index:
lebedev.ri added inline comments.
Comment at: clang-tidy/readability/IsolateDeclCheck.cpp:200
+
+ diag(WholeDecl->getBeginLoc(), "make only one declaration per statement")
+ << FixItHint::CreateReplacement(WholeDecl->getSourceRange(),
Replacement);
I think
JonasToth added a comment.
Yes, do you think it should be included in the diag?
Am 13.09.2018 um 22:09 schrieb Roman Lebedev via Phabricator:
> lebedev.ri added inline comments.
>
>
> Comment at: clang-tidy/readability/IsolateDeclCheck.cpp:200
> +
> + diag(WholeDecl->getBeg
lebedev.ri added a comment.
In https://reviews.llvm.org/D51949#1233951, @JonasToth wrote:
> Yes, do you think it should be included in the diag?
Yes, please :) Else, the message seems a bit too empty.
I **don't** think it should point (via `NOTE:`) at the each decl though.
> Am 13.09.2018 um 2
ormris created this revision.
ormris added reviewers: dblaikie, aprantl, probinson, JDevlieghere, clayborg,
jingham.
This patch adds the associated template parameters to the DWARF name attribute
of all template variable specializations, mirroring how they are referenced in
the source code.
Orig
calixte added inline comments.
Comment at: include/clang/Driver/CC1Options.td:236
+def coverage_exclude_EQ : Joined<["-"], "coverage-exclude=">,
+ Alias;
def coverage_exit_block_before_body : Flag<["-"],
"coverage-exit-block-before-body">,
vsk wrote:
> Have yo
sammccall added a comment.
(Just checking - IIUC neither of you are asking for changes, and this is
waiting on review of the latest snapshot?)
Repository:
rC Clang
https://reviews.llvm.org/D51921
___
cfe-commits mailing list
cfe-commits@lists.ll
ldionne added inline comments.
Comment at: clang/lib/Sema/SemaTemplateInstantiateDecl.cpp:1268
+ // Instantiate the attributes on both the template declaration and the
associated record declaration.
+ SemaRef.InstantiateAttrsForDecl(TemplateArgs, Pattern, Inst, LateAttrs,
Sta
ldionne updated this revision to Diff 165375.
ldionne added a comment.
Drop support for partial specializations and explicit specializations.
Address comments by Erik.
Repository:
rC Clang
https://reviews.llvm.org/D51997
Files:
clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
clang/test/Se
This revision was automatically updated to reflect the committed changes.
Closed by commit rL342177: Diagnose likely typos in #include directives.
(authored by rsmith, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D51333?vs=165361&id=
Author: rsmith
Date: Thu Sep 13 14:10:08 2018
New Revision: 342177
URL: http://llvm.org/viewvc/llvm-project?rev=342177&view=rev
Log:
Diagnose likely typos in #include directives.
Summary:
When someone writes
#include ""
or
#include " some_file "
the compiler returns "file not fuond..." wi
kristina added a comment.
Fine with me, I wasn't really suggesting a revision just making a remark more
or less. Sorry if it came off that way.
Repository:
rC Clang
https://reviews.llvm.org/D51921
___
cfe-commits mailing list
cfe-commits@lists.l
ldionne added a comment.
I created https://bugs.llvm.org/show_bug.cgi?id=38942 to keep track of the
problem for partial specializations and explicit specializations. Moving
forward with this patch will allow me to land the `no_extern_template`
attribute.
Repository:
rC Clang
https://review
rsmith added a comment.
Thank you!
Comment at: include/clang/Basic/DiagnosticASTKinds.td:172
"non-trivially-copyable type %1">;
+def note_constexpr_memcpy_incompletetype : Note<
+ "cannot constant evaluate '%select{memcpy|memmove}0' between objects of "
Nit
Author: rsmith
Date: Thu Sep 13 15:47:33 2018
New Revision: 342192
URL: http://llvm.org/viewvc/llvm-project?rev=342192&view=rev
Log:
Fix crash on call to __builtin_memcpy with a null pointer to an
incomplete type.
Also improve the diagnostics for similar situations.
Modified:
cfe/trunk/inclu
aaronpuchert planned changes to this revision.
aaronpuchert added a comment.
This doesn't work with loops yet:
void relockLoop() {
RelockableExclusiveMutexLock scope(&mu);
while (b) {
scope.Unlock(); // releasing mutex 'scope' that was not held
scope.Lock(); // acquirin
erik.pilkington added a comment.
Looks good to me! @rsmith should probably have the final word though. Thanks
for fixing this.
Repository:
rC Clang
https://reviews.llvm.org/D51997
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://li
zhaomo added inline comments.
Comment at: clang/lib/CodeGen/ItaniumCXXABI.cpp:939
+/// We should only interleave vtables when the module has the hidden
+/// LTO visibility, cfi-vcall is enabled and EnableVTableInterleaving
vlad.tsyrklevich wrote:
> doxygen comm
Author: vedantk
Date: Thu Sep 13 16:28:25 2018
New Revision: 342194
URL: http://llvm.org/viewvc/llvm-project?rev=342194&view=rev
Log:
[Sema] Remove location from implicit capture init expr
A lambda's closure is initialized when the lambda is declared. For
implicit captures, the initialization cod
This revision was automatically updated to reflect the committed changes.
Closed by commit rC342194: [Sema] Remove location from implicit capture init
expr (authored by vedantk, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D50927?vs=162918&id=165400#toc
Repository:
rL LL
This revision was automatically updated to reflect the committed changes.
Closed by commit rL342194: [Sema] Remove location from implicit capture init
expr (authored by vedantk, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D50927?vs=
zhaomo added inline comments.
Comment at: compiler-rt/test/cfi/lit.cfg:33
dso = '-fsanitize-cfi-cross-dso -fvisibility=default '
+ interleave = '-Xclang -enable-vtable-interleaving
-Wl,-plugin-opt,-enable-vtable-interleaving -Wl,-plugin-opt=save-temps
-fno-sanitize=cfi-mfca
vsk created this revision.
vsk added reviewers: rsmith, erichkeane.
When it's not possible to initialize an implicit capture, add a note
pointing to the first use of the captured variable.
Example (the `note` is new):
lambda-expressions.cpp:81:15: error: no matching constructor for initializatio
rsmith added inline comments.
Comment at: clang/lib/Sema/SemaLambda.cpp:1427
if (RefResult.isInvalid())
return ExprError();
Expr *Ref = RefResult.get();
This early exit leaves your CodeSynthesisContext on the stack. Consider using
RAII?
Author: vedantk
Date: Thu Sep 13 16:50:20 2018
New Revision: 342195
URL: http://llvm.org/viewvc/llvm-project?rev=342195&view=rev
Log:
Update a clang-tidy test for r342194
The location of implicit captures has changed. Update a use-after-move
checker test to reflect that.
This fixes a bot failure
vsk updated this revision to Diff 165403.
vsk marked an inline comment as done.
https://reviews.llvm.org/D52064
Files:
clang/include/clang/Basic/DiagnosticSemaKinds.td
clang/include/clang/Sema/Sema.h
clang/lib/Frontend/FrontendActions.cpp
clang/lib/Sema/SemaLambda.cpp
clang/lib/Sema/Sem
vsk added inline comments.
Comment at: clang/test/SemaCXX/lambda-expressions.cpp:87
+(void)^{ // expected-error@+1 {{no matching constructor for initialization
of 'const G'}}
+ return [=]{ // expected-error@+1 {{no matching constructor for
initialization of 'const G'}}
NoQ added a comment.
What do you think about the following approach:
1. Define a `lit` substitution `%diff_plist` that resolves to `diff -u -w -I
"/" -I ".:" -I "version" -`.
2. Update these flags in your downstream fork.
It's kinda cool because it'll allow us to test exactly what we want in th
Author: vedantk
Date: Thu Sep 13 17:16:37 2018
New Revision: 342196
URL: http://llvm.org/viewvc/llvm-project?rev=342196&view=rev
Log:
Relax alignment assumptions in a test after r342194
Don't hardcode align 8.
Fixes a bot failure:
http://lab.llvm.org:8011/builders/clang-cmake-armv8-full/builds/6
steveire accepted this revision.
steveire added a comment.
This revision is now accepted and ready to land.
Thanks!
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D51865
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Looks great to me and pretty trivial, should somebody who's more familiar with
this code take a look?
Comment at: clang/lib/AST/StmtPrinter.cpp:72
PrintingPolicy Policy;
+
Author: maskray
Date: Thu Sep 13 17:56:11 2018
New Revision: 342198
URL: http://llvm.org/viewvc/llvm-project?rev=342198&view=rev
Log:
[clangd] Fix TUScheduler typos
Modified:
clang-tools-extra/trunk/clangd/TUScheduler.cpp
clang-tools-extra/trunk/clangd/TUScheduler.h
Modified: clang-tools
rsmith added inline comments.
Comment at: clang/include/clang/Driver/CC1Options.td:356-357
+HelpText<"Enable VTable interleaving">;
+def disable_vtable_interleaving : Flag<["-"], "disable-vtable-interleaving">,
+HelpText<"Disable VTable interleaving">;
//===
Author: rtrieu
Date: Thu Sep 13 18:15:28 2018
New Revision: 342199
URL: http://llvm.org/viewvc/llvm-project?rev=342199&view=rev
Log:
[ODRHash] Fix early exit that skipped code.
There is a bit of code at the end of AddDeclaration that should be run on
every exit of the function. However, there wa
glaubitz created this revision.
glaubitz added reviewers: kristina, dschuff.
Herald added subscribers: cfe-commits, nemanjai.
On powerpc-linux-gnuspe, the header files are located in their
own include directory named /usr/lib/powerpc-linux-gnuspe,
so add this directory to PPCMultiarchIncludeDirs.
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Looks as expected. I hope that assertion does actually hold. Should AST folks
have a look as well?
Comment at: clang/lib/AST/Stmt.cpp:310
+ return *Out / alignof(Stmt);
+
+}
dberris updated this revision to Diff 165407.
dberris retitled this revision from "[XRay][clang] Always emit XRay attributes
for LLVM IR" to "[XRay][clang] Emit "never-instrument" attribute".
dberris added a comment.
Retitle, add different test case for -fno-xray-instrument.
https://reviews.llv
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Looks good i guess. I'm glad this is sorted out^^
https://reviews.llvm.org/D51057
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.
NoQ added a comment.
Hmm, so we're reporting the same region twice in the same bug report through
different field chains, right? Why is a state trait necessary here? Maybe just
de-duplicate them locally before throwing the report?
https://reviews.llvm.org/D51531
Author: dberris
Date: Thu Sep 13 18:59:12 2018
New Revision: 342200
URL: http://llvm.org/viewvc/llvm-project?rev=342200&view=rev
Log:
[XRay][clang] Emit "never-instrument" attribute
Summary:
Before this change, we only emit the XRay attributes in LLVM IR when the
-fxray-instrument flag is provide
This revision was automatically updated to reflect the committed changes.
Closed by commit rC342200: [XRay][clang] Emit "never-instrument"
attribute (authored by dberris, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D52015?vs=165407&id=165408#toc
Repository:
rC Clang
ht
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Hmm, i guess it's necessary to de-duplicate across multiple reports as well.
https://reviews.llvm.org/D51531
___
cfe-commits mailing list
cfe-commits@l
101 - 200 of 210 matches
Mail list logo
