Author: akirtzidis
Date: Sun Feb 7 13:28:36 2016
New Revision: 260048
URL: http://llvm.org/viewvc/llvm-project?rev=260048&view=rev
Log:
[Frontend] Make the memory management of FrontendAction pointers explicit by
using unique_ptr.
Modified:
cfe/trunk/include/clang/ARCMigrate/ARCMTActions.h
;
> http://reviews.llvm.org/D4313 <http://reviews.llvm.org/D4313>
> http://reviews.llvm.org/D4312 <http://reviews.llvm.org/D4312>
>
> On Sun, Feb 7, 2016 at 11:28 AM, Argyrios Kyrtzidis via cfe-commits
> mailto:cfe-commits@lists.llvm.org>> wrote:
> Author: akirtz
Author: akirtzidis
Date: Tue Feb 9 13:07:21 2016
New Revision: 260254
URL: http://llvm.org/viewvc/llvm-project?rev=260254&view=rev
Log:
[libclang] indexing: handle 'TopLevelDeclInObjCContainers' at the point where
they are reported.
It isn't much benefit and doesn't worth the complexity to try
Author: akirtzidis
Date: Tue Feb 9 13:07:19 2016
New Revision: 260253
URL: http://llvm.org/viewvc/llvm-project?rev=260253&view=rev
Log:
[libclang] indexing: for a synthesized property reference have the parent be
the ObjC implementation decl.
Modified:
cfe/trunk/tools/libclang/IndexingConte
Author: akirtzidis
Date: Tue Feb 9 13:07:07 2016
New Revision: 260250
URL: http://llvm.org/viewvc/llvm-project?rev=260250&view=rev
Log:
[libclang] indexing: Have the semantic container of synthesized ObjC
getter/setter methods be the implementation decl.
Matches the behavior of other ObjC metho
Author: akirtzidis
Date: Tue Feb 9 13:07:13 2016
New Revision: 260251
URL: http://llvm.org/viewvc/llvm-project?rev=260251&view=rev
Log:
[ASTUnit] Change the parameter of ASTUnit::LoadFromCompilerInvocationAction to
accept a more general FrontendAction.
Modified:
cfe/trunk/include/clang/Fron
Author: akirtzidis
Date: Tue Feb 9 13:07:16 2016
New Revision: 260252
URL: http://llvm.org/viewvc/llvm-project?rev=260252&view=rev
Log:
[Frontend] Handle ASTConsumer::shouldSkipFunctionBody via the MultiplexConsumer.
Modified:
cfe/trunk/include/clang/Frontend/MultiplexConsumer.h
cfe/trun
Author: akirtzidis
Date: Tue Feb 9 13:07:24 2016
New Revision: 260255
URL: http://llvm.org/viewvc/llvm-project?rev=260255&view=rev
Log:
[libclang] indexing: make sure to not visit init-list expressions twice.
Modified:
cfe/trunk/test/Index/index-refs.cpp
cfe/trunk/tools/libclang/IndexBod
Author: akirtzidis
Date: Fri Feb 12 17:30:07 2016
New Revision: 260762
URL: http://llvm.org/viewvc/llvm-project?rev=260762&view=rev
Log:
[index] Add llvm/Support/DataTypes.h header to fix build failures in the bots.
Modified:
cfe/trunk/include/clang/Index/IndexSymbol.h
Modified: cfe/trunk/in
Author: akirtzidis
Date: Fri Feb 12 19:24:19 2016
New Revision: 260785
URL: http://llvm.org/viewvc/llvm-project?rev=260785&view=rev
Log:
[RecursiveASTVisitor] Introduce dataTraverseStmtPre()/dataTraverseStmtPost() to
allow clients to do before/after actions during data recursive visitation.
This
Author: akirtzidis
Date: Fri Feb 12 23:17:15 2016
New Revision: 260807
URL: http://llvm.org/viewvc/llvm-project?rev=260807&view=rev
Log:
[index] Change some default parameters to fix an MSVC ICE.
Many thanks to Yunzhong Gao for tracking this down!
Modified:
cfe/trunk/lib/Index/IndexingContex
Feb 13, 2016, at 7:48 AM, Renato Golin wrote:
>
> On 12 February 2016 at 23:11, Argyrios Kyrtzidis via cfe-commits
> wrote:
>> Author: akirtzidis
>> Date: Fri Feb 12 17:10:59 2016
>> New Revision: 260760
>>
>> URL: http://llvm.org/viewvc/llvm-projec
8 AM, Renato Golin wrote:
>>
>> On 12 February 2016 at 23:11, Argyrios Kyrtzidis via cfe-commits
>> wrote:
>>> Author: akirtzidis
>>> Date: Fri Feb 12 17:10:59 2016
>>> New Revision: 260760
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?
v7-a15/stage1/tools/clang/clang.order:
>> file not recognized: File truncated
>>
>> I thought it was some build SNAFU, do you have some hint on what I need to
>> do to fix this ?
>>
>>> On Feb 13, 2016, at 7:48 AM, Renato Golin wrote:
>>>
>&
Author: akirtzidis
Date: Sat Feb 13 15:46:50 2016
New Revision: 260833
URL: http://llvm.org/viewvc/llvm-project?rev=260833&view=rev
Log:
[AST] Add a print() method in DeclarationName that accepts a PrintingPolicy.
Modified:
cfe/trunk/include/clang/AST/DeclarationName.h
cfe/trunk/lib/AST/D
Author: akirtzidis
Date: Sun Feb 14 00:39:11 2016
New Revision: 260842
URL: http://llvm.org/viewvc/llvm-project?rev=260842&view=rev
Log:
[index] Enhance c-index-test tool and have it link and test the clangIndex
library directly.
Added:
cfe/trunk/test/Index/Core/
cfe/trunk/test/Index/Cor
Author: akirtzidis
Date: Sun Feb 14 00:39:03 2016
New Revision: 260841
URL: http://llvm.org/viewvc/llvm-project?rev=260841&view=rev
Log:
[index] Allow calling createIndexingAction() without passing another action to
wrap over.
Modified:
cfe/trunk/include/clang/Index/IndexingAction.h
cfe/
Author: akirtzidis
Date: Sun Feb 14 00:53:20 2016
New Revision: 260843
URL: http://llvm.org/viewvc/llvm-project?rev=260843&view=rev
Log:
[index] Fix gcc builds.
Modified:
cfe/trunk/include/clang/Index/IndexingAction.h
cfe/trunk/tools/c-index-test/core_main.cpp
Modified: cfe/trunk/include
Author: akirtzidis
Date: Sun Feb 14 01:08:31 2016
New Revision: 260844
URL: http://llvm.org/viewvc/llvm-project?rev=260844&view=rev
Log:
[c-index-test] Fix a gcc build error.
Modified:
cfe/trunk/tools/c-index-test/core_main.cpp
Modified: cfe/trunk/tools/c-index-test/core_main.cpp
URL:
http:
nt modules.
>
> I think you should do;
> - Export C++ API in libclang. (But I don't expect it would work for win32)
> - Link libclang as static. Then c-index-test became the test that didn't
> test "libclang".
>
> On Sun, Feb 14, 2016 at 3:43
Thanks!
> On Feb 14, 2016, at 5:18 AM, Benjamin Kramer via cfe-commits
> wrote:
>
> Author: d0k
> Date: Sun Feb 14 07:18:06 2016
> New Revision: 260850
>
> URL: http://llvm.org/viewvc/llvm-project?rev=260850&view=rev
> Log:
> Don't leak the ASTUnit when done with testing.
>
> Found by lsan.
>
Which configuration required them ?
I was under the impression CMake is passing the transitive libdeps.
> On Feb 14, 2016, at 1:19 AM, NAKAMURA Takumi via cfe-commits
> wrote:
>
> Author: chapuni
> Date: Sun Feb 14 03:19:04 2016
> New Revision: 260847
>
> URL: http://llvm.org/viewvc/llvm-proje
Author: akirtzidis
Date: Sun Feb 14 16:30:14 2016
New Revision: 260858
URL: http://llvm.org/viewvc/llvm-project?rev=260858&view=rev
Log:
[index] Factor libclang's functionality to determing the mangled name of
symbols into the clangIndex library.
Added:
cfe/trunk/include/clang/Index/CodegenN
Author: akirtzidis
Date: Sun Feb 14 16:38:38 2016
New Revision: 260861
URL: http://llvm.org/viewvc/llvm-project?rev=260861&view=rev
Log:
[test/Index] Set a specific target for the test.
Modified:
cfe/trunk/test/Index/Core/index-source.m
Modified: cfe/trunk/test/Index/Core/index-source.m
URL:
Author: akirtzidis
Date: Sun Feb 14 19:32:36 2016
New Revision: 260866
URL: http://llvm.org/viewvc/llvm-project?rev=260866&view=rev
Log:
[AST/index] Introduce an option 'SuppressTemplateArgsInCXXConstructors' in
printing policy.
Enable it for USRs and names when indexing.
Forward references can
Author: akirtzidis
Date: Mon Feb 15 23:39:33 2016
New Revision: 260937
URL: http://llvm.org/viewvc/llvm-project?rev=260937&view=rev
Log:
[Frontend] Make sure WrapperFrontendAction updates CurrentInput after calling
BeginSourceFileAction.
I don't have a test case to add unfortunately.
Modified:
Author: akirtzidis
Date: Thu Feb 18 17:08:36 2016
New Revision: 261274
URL: http://llvm.org/viewvc/llvm-project?rev=261274&view=rev
Log:
[ASTImporter] Implement missing VisitAccessSpecDecl function in ASTImporter
class.
Patch by Elisavet Sakellari!
Modified:
cfe/trunk/lib/AST/ASTImporter.cp
akyrtzi added a subscriber: akyrtzi.
akyrtzi accepted this revision.
akyrtzi added a reviewer: akyrtzi.
akyrtzi added a comment.
This revision is now accepted and ready to land.
LGTM, committed in r261274.
http://reviews.llvm.org/D16923
___
cfe-comm
akyrtzi added a subscriber: akyrtzi.
akyrtzi added a comment.
Doesn't this mean that _all_ of the Importer.FromDiag() calls will be ignored
by VerifyDiagnosticConsumer ? Why specifically change only this two and what
are we going to do with the others ?
This seems more like needing a fix higher
akyrtzi added a subscriber: akyrtzi.
akyrtzi added a comment.
If I apply just the test changes without the rest of the changes, I don't see
any failures, so I'm not sure these are effective tests.
http://reviews.llvm.org/D17029
___
cfe-commits mail
akyrtzi added a comment.
I'm not sure removing all the Importer.FromDiag() calls is the right thing. You
are changing
Importer.FromDiag(D->getLocation(), diag::err_unsupported_ast_node)
to
Importer.ToDiag(SourceLocation(), diag::err_unsupported_ast_node)
and you lose the source location infor
akyrtzi added a comment.
Could you check if you can decouple VerifyDiagnosticConsumer from depending on
a particular DiagnosticsEngine ?
http://reviews.llvm.org/D17026
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cg
Author: akirtzidis
Date: Sat Feb 20 14:34:55 2016
New Revision: 261445
URL: http://llvm.org/viewvc/llvm-project?rev=261445&view=rev
Log:
[c-index-test] CMake: When installing c-index-test to a different prefix
directory, add an rpath so that
it can find libclang.
Modified:
cfe/trunk/tools/c-
Could you also check performance of creating a PCH file out of Cocoa.h ?
> On May 19, 2016, at 1:47 PM, Vedant Kumar wrote:
>
> vsk added a comment.
>
> I discussed this bug with Argyrios off-list, who lgtm'd on the condition that
> it doesn't introduce a performance regression. He suggested p
Author: akirtzidis
Date: Thu Mar 31 15:18:22 2016
New Revision: 265042
URL: http://llvm.org/viewvc/llvm-project?rev=265042&view=rev
Log:
[index] Fix regression where ObjC method declarations may mistakenly get
indexed as definition.
rdar://25372906
Modified:
cfe/trunk/lib/Index/IndexingCont
Author: akirtzidis
Date: Fri Apr 22 02:21:10 2016
New Revision: 267117
URL: http://llvm.org/viewvc/llvm-project?rev=267117&view=rev
Log:
[index] Add a SymbolSubKind for an ObjC unit test.
Added:
cfe/trunk/test/Index/Core/index-subkinds.m
Modified:
cfe/trunk/include/clang/Index/IndexSymbol
Author: akirtzidis
Date: Fri Apr 22 02:21:04 2016
New Revision: 267116
URL: http://llvm.org/viewvc/llvm-project?rev=267116&view=rev
Log:
[index] Change SymbolCXXTemplateKind to a 'SymbolSubKinds' bitset.
This provides a more general and flexible way to annotate special symbols.
Modified:
cfe
Author: akirtzidis
Date: Fri Apr 22 02:21:16 2016
New Revision: 267118
URL: http://llvm.org/viewvc/llvm-project?rev=267118&view=rev
Log:
[index] Add SymbolSubKinds for ObjC IB annotations.
Modified:
cfe/trunk/include/clang/Index/IndexSymbol.h
cfe/trunk/lib/Index/IndexSymbol.cpp
cfe/tr
Author: akirtzidis
Date: Thu Jun 30 20:17:02 2016
New Revision: 274314
URL: http://llvm.org/viewvc/llvm-project?rev=274314&view=rev
Log:
[CodeCompletion] Allow system headers providing private symbols with a single
underscore.
rdar://24677150
Modified:
cfe/trunk/lib/Sema/SemaCodeComplete.cp
Author: akirtzidis
Date: Fri Jul 1 14:10:54 2016
New Revision: 274378
URL: http://llvm.org/viewvc/llvm-project?rev=274378&view=rev
Log:
[libclang] Sync-up the way top-level decls in an ASTUnit are handled with how
decls in a DeclContext are handled.
rdar://19775013
Modified:
cfe/trunk/test
Author: akirtzidis
Date: Wed Jul 13 15:35:26 2016
New Revision: 275313
URL: http://llvm.org/viewvc/llvm-project?rev=275313&view=rev
Log:
[PCH/preamble] Make sure that if the preamble/PCH was serialized with errors
that we set diagnostic engine state appropriately.
Otherwise there can be a crash
Author: akirtzidis
Date: Wed Jul 13 17:03:47 2016
New Revision: 275324
URL: http://llvm.org/viewvc/llvm-project?rev=275324&view=rev
Log:
[test] Add 'env' to fix test failures in windows bots.
Modified:
cfe/trunk/test/Index/pch-warn-as-error-code-split.cpp
Modified: cfe/trunk/test/Index/pch-w
Author: akirtzidis
Date: Thu Jul 14 15:21:16 2016
New Revision: 275466
URL: http://llvm.org/viewvc/llvm-project?rev=275466&view=rev
Log:
[arcmt/objcmt] Fix ParentMap crash with invalid code.
rdar://22489560
Added:
cfe/trunk/test/ARCMT/objcmt-invalid-code.mm
cfe/trunk/test/ARCMT/objcmt-in
Author: akirtzidis
Date: Fri Jul 15 13:11:27 2016
New Revision: 275589
URL: http://llvm.org/viewvc/llvm-project?rev=275589&view=rev
Log:
[objcmt] Don't add an #import of Foundation unnecessarily, if the NS_ENUM macro
is already defined.
Modified:
cfe/trunk/lib/ARCMigrate/ObjCMT.cpp
cfe/t
Author: akirtzidis
Date: Fri Jul 15 13:11:33 2016
New Revision: 275590
URL: http://llvm.org/viewvc/llvm-project?rev=275590&view=rev
Log:
[AST] Keep track of the left brace source location of a tag decl.
This is useful for source modification tools. There will be a follow-up commit
using it.
Mod
Author: akirtzidis
Date: Fri Jul 15 14:22:34 2016
New Revision: 275600
URL: http://llvm.org/viewvc/llvm-project?rev=275600&view=rev
Log:
[objcmt] Fix a buffer overflow crash than can occur while modernizing enums.
Note that due to the nature of the crash it requires libgmalloc or asan for it
to
> On Jul 15, 2016, at 12:58 PM, Ben Langmuir wrote:
>
>>
>> On Jul 15, 2016, at 12:22 PM, Argyrios Kyrtzidis via cfe-commits
>> wrote:
>>
>> Author: akirtzidis
>> Date: Fri Jul 15 14:22:34 2016
>> New Revision: 275600
>>
>>
Author: akirtzidis
Date: Fri Jul 15 15:40:24 2016
New Revision: 275609
URL: http://llvm.org/viewvc/llvm-project?rev=275609&view=rev
Log:
[test/objcmt] Add a follow-up test case for r275600.
Modified:
cfe/trunk/test/ARCMT/whitelisted/header1.h
cfe/trunk/test/ARCMT/whitelisted/header1.h.res
> On Jul 15, 2016, at 1:03 PM, Argyrios Kyrtzidis wrote:
>
>>
>> On Jul 15, 2016, at 12:58 PM, Ben Langmuir > <mailto:blangm...@apple.com>> wrote:
>>
>>>
>>> On Jul 15, 2016, at 12:22 PM, Argyrios Kyrtzidis via cfe-commits
>>
Author: akirtzidis
Date: Fri Jul 15 17:18:19 2016
New Revision: 275630
URL: http://llvm.org/viewvc/llvm-project?rev=275630&view=rev
Log:
[index] Create different USR if a property is a class property.
Avoids USR conflicts between class & instance properties of the same name.
Modified:
cfe/tr
Author: akirtzidis
Date: Mon Feb 29 01:55:51 2016
New Revision: 262205
URL: http://llvm.org/viewvc/llvm-project?rev=262205&view=rev
Log:
[index] Use ',' to separate symbol roles when printing.
Modified:
cfe/trunk/lib/Index/IndexSymbol.cpp
cfe/trunk/test/Index/Core/index-source.cpp
cfe
Author: akirtzidis
Date: Mon Feb 29 01:56:07 2016
New Revision: 262208
URL: http://llvm.org/viewvc/llvm-project?rev=262208&view=rev
Log:
[index] Print and test module import references.
Added:
cfe/trunk/test/Index/Core/Inputs/
cfe/trunk/test/Index/Core/Inputs/module/
cfe/trunk/test/In
Author: akirtzidis
Date: Mon Feb 29 01:55:55 2016
New Revision: 262206
URL: http://llvm.org/viewvc/llvm-project?rev=262206&view=rev
Log:
[AST/RecursiveASTVisitor] Correction so that dataTraverseStmtPost will get
called after the statement has been visited.
Fixes the indexing client of this.
Mod
Author: akirtzidis
Date: Mon Feb 29 01:56:00 2016
New Revision: 262207
URL: http://llvm.org/viewvc/llvm-project?rev=262207&view=rev
Log:
[index] Add a caller relation for a call reference.
Modified:
cfe/trunk/include/clang/Index/IndexSymbol.h
cfe/trunk/lib/Index/IndexBody.cpp
cfe/trun
I'm just the first person to bother bisecting this...
> Maybe the test just needs to be pared down some?
>
> Please advise.
>
> Regards,
> -Justin
>
> On Sat, Feb 13, 2016 at 12:47 PM, Argyrios Kyrtzidis via cfe-commits
> wrote:
>> I guess refreshing the build
Mon, Feb 29, 2016 at 3:03 PM, Argyrios Kyrtzidis wrote:
>> Is this still an issue after r260785 ?
>> Could you provide a stack trace ?
>>
>>> On Feb 29, 2016, at 2:27 PM, Justin Lebar wrote:
>>>
>>> Hi, I think this broke clang/test/Index/in
eStmt(clang::Stmt*,
>>> llvm::SmallVectorImpl>> llvm::PointerLikeTypeTraits,
>>> llvm::PointerIntPairInfo>> llvm::PointerLikeTypeTraits > > >*) () from
>>> /usr/local/google/home/jlebar/code/llvm-complete/release/bin/../lib/libclang.so.3.9
>
Author: akirtzidis
Date: Mon Feb 29 20:46:32 2016
New Revision: 262290
URL: http://llvm.org/viewvc/llvm-project?rev=262290&view=rev
Log:
[index] Fix issue where data visitation was disabled with C++ operator call
expressions, during indexing.
Modified:
cfe/trunk/lib/Index/IndexBody.cpp
Modi
sitor<(anonymous
>>>> namespace)::BodyIndexer>::dataTraverseNode(clang::Stmt*,
>>>> llvm::SmallVectorImpl>>> llvm::PointerLikeTypeTraits,
>>>> llvm::PointerIntPairInfo>>> llvm::PointerLikeTypeTraits > > >*) () from
>>>> /
asan was also hitting stack overflow and I’m going to try re-enabling
for that test.
> On Feb 29, 2016, at 8:10 PM, David Blaikie wrote:
>
> Does this change any behavior? (missing test case?)
>
> On Mon, Feb 29, 2016 at 6:46 PM, Argyrios Kyrtzidis via cfe-commits
Author: akirtzidis
Date: Mon Feb 29 22:40:20 2016
New Revision: 262295
URL: http://llvm.org/viewvc/llvm-project?rev=262295&view=rev
Log:
[test] After r262290 reducing stack usage, try re-enabling asan on the related
test.
Modified:
cfe/trunk/test/Index/index-many-call-ops.cpp
Modified: cfe/
t;> llvm::PointerLikeTypeTraits > > >*) () from
>>>>>>> /usr/local/google/home/jlebar/code/llvm-complete/release/bin/../lib/libclang.so.3.9
>>>>>>> #17 0x751a880e in clang::RecursiveASTVisitor<(anonymous
>>>>&g
akyrtzi added a comment.
We should not bake-in filtering into the clang CodeComplete API. What kind of
filtering to use should be on a higher level, different clients may want
different filtering, e.g. prefix filtering, fuzzy filtering, etc.
Repository:
rL LLVM
http://reviews.llvm.org/D1782
akyrtzi added a comment.
In http://reviews.llvm.org/D17820#366638, @milianw wrote:
> But, we currently always request code completion at a word start boundary so
> nothing would change for us. That said, I see how this patch could be seen as
> a breaking behavior change, and thus should probabl
Author: akirtzidis
Date: Wed Mar 2 23:33:54 2016
New Revision: 262584
URL: http://llvm.org/viewvc/llvm-project?rev=262584&view=rev
Log:
[index] Report references of ObjC super class/protocols in interfaces and
protocols.
Modified:
cfe/trunk/lib/Index/IndexDecl.cpp
cfe/trunk/test/Index/C
Author: akirtzidis
Date: Wed Mar 2 23:33:48 2016
New Revision: 262583
URL: http://llvm.org/viewvc/llvm-project?rev=262583&view=rev
Log:
[test] Re-enable asan on the test, stack usage should be lower.
Modified:
cfe/trunk/test/Index/index-many-logical-ops.c
Modified: cfe/trunk/test/Index/inde
Mind if we keep -fmodule-implementation-of as an alias so that we can gradually
transition to -fmodule-name ?
> On Feb 19, 2016, at 2:25 PM, Richard Smith via cfe-commits
> wrote:
>
> Author: rsmith
> Date: Fri Feb 19 16:25:36 2016
> New Revision: 261372
>
> URL: http://llvm.org/viewvc/llvm-p
Author: akirtzidis
Date: Thu Mar 3 22:24:32 2016
New Revision: 262686
URL: http://llvm.org/viewvc/llvm-project?rev=262686&view=rev
Log:
[index] Ignore ObjCTypeParamDecls during indexing.
Modified:
cfe/trunk/lib/Index/IndexingContext.cpp
cfe/trunk/test/Index/Core/index-source.m
Modified:
Author: akirtzidis
Date: Fri Mar 4 01:17:48 2016
New Revision: 262694
URL: http://llvm.org/viewvc/llvm-project?rev=262694&view=rev
Log:
[index] Include parameter types in the USRs for C functions marked with
'overloadable' attribute.
Modified:
cfe/trunk/lib/Index/USRGeneration.cpp
cfe/t
Author: akirtzidis
Date: Fri Mar 4 01:17:43 2016
New Revision: 262693
URL: http://llvm.org/viewvc/llvm-project?rev=262693&view=rev
Log:
[index] In ObjC++ handle objc type parameters for function USRs.
Added:
cfe/trunk/test/Index/Core/index-source.mm
Modified:
cfe/trunk/lib/Index/USRGener
Author: akirtzidis
Date: Fri Mar 4 01:17:53 2016
New Revision: 262695
URL: http://llvm.org/viewvc/llvm-project?rev=262695&view=rev
Log:
[index] Distinguish USRs of anonymous enums by using their first enumerator.
rdar://24609949.
Modified:
cfe/trunk/lib/Index/IndexingContext.cpp
cfe/tru
Author: akirtzidis
Date: Tue Mar 8 20:12:40 2016
New Revision: 262984
URL: http://llvm.org/viewvc/llvm-project?rev=262984&view=rev
Log:
[index] Fix assertion hit when indexing re-declarations of built-in functions.
Modified:
cfe/trunk/lib/Index/IndexingContext.cpp
cfe/trunk/test/Index/Co
Author: akirtzidis
Date: Tue Mar 8 20:12:46 2016
New Revision: 262985
URL: http://llvm.org/viewvc/llvm-project?rev=262985&view=rev
Log:
[index] libclang: Make sure to treat forward ObjC protocols as ObjCProtocolRef
declarations, and fix related crash.
rdar://25035376
Modified:
cfe/trunk/li
Author: akirtzidis
Date: Tue Mar 8 20:53:12 2016
New Revision: 262991
URL: http://llvm.org/viewvc/llvm-project?rev=262991&view=rev
Log:
[index] Add a message for the assertion, NFC.
Modified:
cfe/trunk/lib/Index/IndexingContext.cpp
Modified: cfe/trunk/lib/Index/IndexingContext.cpp
URL:
htt
> On Mar 8, 2016, at 6:28 PM, Ben Langmuir wrote:
>
>>
>> On Mar 8, 2016, at 6:12 PM, Argyrios Kyrtzidis via cfe-commits
>> wrote:
>>
>> Author: akirtzidis
>> Date: Tue Mar 8 20:12:40 2016
>> New Revision: 262984
>>
>> URL: ht
If not addressing the issue with '-fmodule-name' could you revert removal of
original ‘-fmodule-implementation-of’ since we are depending on it ?
> On Mar 3, 2016, at 6:13 PM, Argyrios Kyrtzidis via cfe-commits
> wrote:
>
> Hey Richard,
>
> -fmodule-name emits b
Author: akirtzidis
Date: Wed Mar 16 23:28:19 2016
New Revision: 263689
URL: http://llvm.org/viewvc/llvm-project?rev=263689&view=rev
Log:
[index] Make sure that declarations of builtin functions are indexed.
rdar://25154630
Modified:
cfe/trunk/lib/Index/IndexingContext.cpp
cfe/trunk/test/
The long-term intention of this was to separate user modules vs system modules,
so that you could have user modules generated in the project-specific build
directory while system modules would be in the global directory shared across
projects.
That way when you deleted the project build director
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
Author: akirtzidis
Date: Fri Jan 15 18:20:02 2016
New Revision: 257968
URL: http://llvm.org/viewvc/llvm-project?rev=257968&view=rev
Log:
[libclang] Introduce APIs for evaluating a cursor and checking if a macro is
builtin/function.
rdar://24091595
Added:
cfe/trunk/test/Index/evaluate-cursor
Author: akirtzidis
Date: Fri Jan 15 21:01:20 2016
New Revision: 257982
URL: http://llvm.org/viewvc/llvm-project?rev=257982&view=rev
Log:
[libclang] Add missing CINDEX_LINKAGE from some new APIs in Index.h.
Modified:
cfe/trunk/include/clang-c/Index.h
Modified: cfe/trunk/include/clang-c/Index.
> On Jan 16, 2016, at 1:15 AM, Ismail Donmez wrote:
>
> On Sat, Jan 16, 2016 at 5:01 AM, Argyrios Kyrtzidis via cfe-commits
> wrote:
>> Author: akirtzidis
>> Date: Fri Jan 15 21:01:20 2016
>> New Revision: 257982
>>
>> URL: http://llvm.org/view
Author: akirtzidis
Date: Fri Jan 29 19:51:20 2016
New Revision: 259287
URL: http://llvm.org/viewvc/llvm-project?rev=259287&view=rev
Log:
[SemaCXX] Fix crash-on-invalid while trying to deduce return type of a lambda.
rdar://22032373
Modified:
cfe/trunk/lib/Sema/SemaStmt.cpp
cfe/trunk/test
Author: akirtzidis
Date: Thu Nov 12 19:46:18 2015
New Revision: 253001
URL: http://llvm.org/viewvc/llvm-project?rev=253001&view=rev
Log:
[CMake] If 'INTERNAL_INSTALL_PREFIX' is set, use it for determining the install
destination of c-index-test and the libclang headers.
Modified:
cfe/trunk/C
.com>> wrote:
> Hi,
>
> On Thu, Nov 12, 2015 at 2:46 AM, Argyrios Kyrtzidis via cfe-commits
> mailto:cfe-commits@lists.llvm.org>> wrote:
> --- cfe/trunk/tools/c-index-test/CMakeLists.txt (original)
> +++ cfe/trunk/tools/c-index-test/CMakeLists.txt Wed Nov 11 18:46:57
gt; ~Aaron
>
> On Wed, Nov 11, 2015 at 7:46 PM, Argyrios Kyrtzidis via cfe-commits
> wrote:
>> Author: akirtzidis
>> Date: Wed Nov 11 18:46:57 2015
>> New Revision: 252836
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=252836&view=rev
>> Log:
Author: akirtzidis
Date: Fri Nov 13 16:41:14 2015
New Revision: 253099
URL: http://llvm.org/viewvc/llvm-project?rev=253099&view=rev
Log:
[CMake] Don't install c-index-test when LLVM_INSTALL_TOOLCHAIN_ONLY=ON.
Modified:
cfe/trunk/tools/c-index-test/CMakeLists.txt
Modified: cfe/trunk/tools/c-i
LGTM.
> On Nov 16, 2015, at 12:32 PM, Ben Craig wrote:
>
> bcraig added a comment.
>
> Ping. Note that the test is basically a copy / paste job, and the new code
> in DataRecursiveASTVisitor.h is a very direct translation from the 'regular'
> RecursiveASTVisitor.h.
>
>
> http://reviews.llv
W00t! That’s awesome Richard!
> On Nov 16, 2015, at 5:10 PM, Richard Smith wrote:
>
> Attached patch makes RAV fully data-recursive when visiting statements,
> except in cases where the derived class could tell the difference (when it
> falls back to a normal recursive walk). The queue represe
Author: akirtzidis
Date: Thu Nov 19 20:24:03 2015
New Revision: 253636
URL: http://llvm.org/viewvc/llvm-project?rev=253636&view=rev
Log:
[CMake] Add a specific 'install-clang-headers' target.
Modified:
cfe/trunk/lib/Headers/CMakeLists.txt
Modified: cfe/trunk/lib/Headers/CMakeLists.txt
URL:
Author: akirtzidis
Date: Thu Nov 19 21:36:21 2015
New Revision: 253645
URL: http://llvm.org/viewvc/llvm-project?rev=253645&view=rev
Log:
[libclang] Make sure to use the raw module format for libclang parsing.
Fixes crash when passing '-gmodules' in the compiler options.
rdar://23588717
Modified:
> On Dec 7, 2015, at 12:16 AM, John McCall wrote:
>
> rjmccall added a reviewer: akyrtzi.
>
>
> Comment at: lib/AST/ASTContext.cpp:5424
> @@ -5410,1 +5423,3 @@
> +// FIXME: need to figure out what this is for __float128
> +case BuiltinType::Float128: return 'K';
>
Hi Bruno,
> On Dec 2, 2015, at 7:29 PM, Bruno Cardoso Lopes
> wrote:
>
> bruno created this revision.
> bruno added reviewers: doug.gregor, akyrtzi.
> bruno added subscribers: cfe-commits, dexonsmith.
>
> Consider the following ObjC++ snippet:
>
> @protocol PA;
> @protocol PB;
>
> @class
akyrtzi added a subscriber: akyrtzi.
akyrtzi added a comment.
Hi Bruno,
http://reviews.llvm.org/D15173
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
For CX_BinaryOperatorKind please change the enumerators to have an explicit
integer assignment; this is to emphasize that they are API contract and should
not change with subsequent changes (someone may inadvertently add a new
enumerator in the middle which will implicitly change all the subsequ
akyrtzi added a subscriber: akyrtzi.
akyrtzi added a comment.
For CX_BinaryOperatorKind please change the enumerators to have an explicit
integer assignment; this is to emphasize that they are API contract and should
not change with subsequent changes (someone may inadvertently add a new
enumer
akyrtzi added a subscriber: akyrtzi.
akyrtzi added a comment.
The patch does not apply cleanly, could you provide a more up-to-date one ?
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cg
The patch does not apply cleanly, could you provide a more up-to-date one ?
> On Dec 6, 2015, at 11:46 PM, guibufolo+l...@gmail.com wrote:
>
> RedX2501 added a comment.
>
> Ping
>
>
> http://reviews.llvm.org/D10834
>
>
>
___
cfe-commits mailing l
akyrtzi added a comment.
The patch causes tests to fail, please look into it.
http://reviews.llvm.org/D10833
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
201 - 300 of 342 matches
Mail list logo
