[PATCH] D55782: Fix isInSystemMacro to handle pasted token

serge-sans-paille marked an inline comment as done. serge-sans-paille added a comment. Herald added a subscriber: llvm-commits. It's in, thanks @rsmith for the review. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55782/new/ https://reviews.llvm.org/D55782

[PATCH] D55782: Fix isInSystemMacro to handle pasted token

This revision was automatically updated to reflect the committed changes. Closed by commit rL352838: Fix isInSystemMacro to handle pasted macros (authored by serge_sans_paille, committed by ). Herald added a project: LLVM. Changed prior to commit: https://reviews.llvm.org/D55782?vs=184275&id=18

[clang-tools-extra] r352837 - [clangd] Fix crash in applyTweak, remove TweakID alias.

Author: sammccall Date: Thu Jan 31 21:41:50 2019 New Revision: 352837 URL: http://llvm.org/viewvc/llvm-project?rev=352837&view=rev Log: [clangd] Fix crash in applyTweak, remove TweakID alias. Strings are complicated, giving them opaque names makes us forget they're complicated. Modified: cla

[PATCH] D57388: [clangd] Implement textDocument/declaration from LSP 3.14

sammccall updated this revision to Diff 184678. sammccall marked 7 inline comments as done. sammccall added a comment. Herald added a project: clang. Address review comments Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57388/new/ https://reviews.ll

[PATCH] D57388: [clangd] Implement textDocument/declaration from LSP 3.14

sammccall added inline comments. Comment at: clangd/ClangdLSPServer.cpp:332 }}, +{"declarationProvider", true}, {"definitionProvider", true}, hokein wrote: > I believe missing this is a LSP protocol bug, vscode uses this ter

[PATCH] D57325: [clangd] Collect macros in static index.

sammccall added a comment. Herald added a project: clang. So I'm not sure this is actually going to be a win. When building google's big static index, we index these symbols and then throw most of them away (a high usage threshold applies). But there's no mechanism here to do that. If we do inde

[PATCH] D57438: [Sema][ObjC] Allow declaring ObjC pointer members in C++ unions under ARC

rjmccall added inline comments. Comment at: lib/Sema/SemaDeclCXX.cpp:7084 + if (FD->getParent()->isUnion() && + shouldDeleteForVariantObjCPtrMember(FD, FieldType)) rjmccall wrote: > I believe the right check for variant-ness is `inUnion()`, not > `FD->ge

[PATCH] D56555: Add Attribute to define nonlazy objc classes

rjmccall accepted this revision. rjmccall added a comment. Thanks! I'd like Aaron to sign off as well before we commit this. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56555/new/ https://reviews.llvm.org/D56555 _

[PATCH] D56555: Add Attribute to define nonlazy objc classes

joedaniels29 updated this revision to Diff 184671. joedaniels29 marked 4 inline comments as done. joedaniels29 added a comment. Herald added a project: clang. Docs changes, formatting. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56555/new/ https://reviews.llvm.

[PATCH] D53076: [analyzer] Enhance ConditionBRVisitor to write out more information

NoQ added inline comments. Comment at: lib/StaticAnalyzer/Core/BugReporterVisitors.cpp:166-173 + // The declaration of the value may rely on a pointer so take its l-value. + if (const auto *DRE = dyn_cast_or_null(CondVarExpr)) { +if (const auto *VD = dyn_cast_or_null(DRE->g

[PATCH] D57032: [SemaCXX] Param diagnostic matches overload logic

modocache added a comment. Thanks for the reviews! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57032/new/ https://reviews.llvm.org/D57032 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.o

[PATCH] D57032: [SemaCXX] Param diagnostic matches overload logic

This revision was automatically updated to reflect the committed changes. Closed by commit rC352831: [SemaCXX] Param diagnostic matches overload logic (authored by modocache, committed by ). Changed prior to commit: https://reviews.llvm.org/D57032?vs=184284&id=184667#toc Repository: rC Clang

r352831 - [SemaCXX] Param diagnostic matches overload logic

Author: modocache Date: Thu Jan 31 19:30:29 2019 New Revision: 352831 URL: http://llvm.org/viewvc/llvm-project?rev=352831&view=rev Log: [SemaCXX] Param diagnostic matches overload logic Summary: Given the following test program: ``` class C { public: int A(int a, int& b); }; int C::A(const in

[PATCH] D57497: [RISCV] Passing -G value to RISCV backend

shiva0217 added a comment. Herald added a project: clang. In D57497#1378511 , @simoncook wrote: > As this mllvm option only affects the creation of ELF objects, do we also > need to add a similar option for the LTO case, as the -G value would have no > e

[PATCH] D57032: [SemaCXX] Param diagnostic matches overload logic

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Thanks, LGTM. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57032/new/ https://reviews.llvm.org/D57032 ___ cfe-c

r352829 - [Sanitizers] UBSan unreachable incompatible with ASan in the presence of `noreturn` calls

Author: yln Date: Thu Jan 31 18:51:00 2019 New Revision: 352829 URL: http://llvm.org/viewvc/llvm-project?rev=352829&view=rev Log: [Sanitizers] UBSan unreachable incompatible with ASan in the presence of `noreturn` calls Summary: UBSan wants to detect when unreachable code is actually reached, so

[PATCH] D57032: [SemaCXX] Param diagnostic matches overload logic

modocache marked an inline comment as done. modocache added a comment. Herald added a project: clang. Friendly ping! Is there anything I can do to improve this patch? I think it's a clear improvement of the diagnostic, with a test case to boot! Repository: rC Clang CHANGES SINCE LAST ACTION

[PATCH] D57438: [Sema][ObjC] Allow declaring ObjC pointer members in C++ unions under ARC

ahatanak updated this revision to Diff 184665. ahatanak marked 3 inline comments as done. ahatanak added a comment. Address review comments. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57438/new/ https://reviews.llvm.org/D57438 Files: include/clang/Basic/Dia

[PATCH] D57438: [Sema][ObjC] Allow declaring ObjC pointer members in C++ unions under ARC

ahatanak added inline comments. Comment at: test/SemaObjCXX/arc-0x.mm:164 +union { + union { // expected-note 2 {{'S1' is implicitly deleted because variant field '' has a non-trivial}} expected-note 4 {{'S1' is implicitly deleted because field '' has a deleted}} +

r352827 - [opaque pointer types] Add a FunctionCallee wrapper type, and use it.

Author: jyknight Date: Thu Jan 31 18:28:03 2019 New Revision: 352827 URL: http://llvm.org/viewvc/llvm-project?rev=352827&view=rev Log: [opaque pointer types] Add a FunctionCallee wrapper type, and use it. Recommit r352791 after tweaking DerivedTypes.h slightly, so that gcc doesn't choke on it, ho

r352824 - [analyzer] [RetainCountChecker] Fix object type for CF/Obj-C bridged casts

Author: george.karpenkov Date: Thu Jan 31 18:13:02 2019 New Revision: 352824 URL: http://llvm.org/viewvc/llvm-project?rev=352824&view=rev Log: [analyzer] [RetainCountChecker] Fix object type for CF/Obj-C bridged casts Having an incorrect type for a cast causes the checker to incorrectly dismiss t

Re: [PATCH] D56611: [clangd] A code action to swap branches of an if statement

Thanks! I suspect shared libraries makes a difference here, because the tweaks use an unusual library type (object library) to force linking. On Fri, Feb 1, 2019 at 3:01 AM Nathan Ridge via Phabricator < revi...@reviews.llvm.org> wrote: > nridge added a comment. > > Fix here: https://reviews.llvm

[PATCH] D45978: dllexport const variables must have external linkage.

rnk added a comment. I'm still not sure this is the best place to make this change, but the functionality is important. There are still unaddressed comments (no need to check MSVCCompatibility, formatting), and I think once those are fixed we can land this. Comment at: test/

[PATCH] D56611: [clangd] A code action to swap branches of an if statement

nridge added a comment. Fix here: https://reviews.llvm.org/D57560 Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56611/new/ https://reviews.llvm.org/D56611 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:

[PATCH] D56611: [clangd] A code action to swap branches of an if statement

nridge added a comment. The complete commands that's failing is: /usr/bin/clang++-8 -fPIC -fvisibility-inlines-hidden -Werror=date-time -Werror=unguarded-availability-new -std=c++11 -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic -Wno-l

[PATCH] D56611: [clangd] A code action to swap branches of an if statement

nridge added a comment. In D56611#1379876 , @sammccall wrote: > Hi Nathan, > What platform is this on? Not seeing it on the buildbots. > Anything unusual in build setup (standalone build, building with shared > libraries, etc)? I'm on Linux, building

[PATCH] D56611: [clangd] A code action to swap branches of an if statement

sammccall added a comment. In D56611#1379785 , @nridge wrote: > This commit is causing clangd to fail to link, with errors like this: > > > tools/clang/tools/extra/clangd/refactor/tweaks/CMakeFiles/obj.clangDaemonTweaks.dir/SwapIfBranches.cpp.o:SwapIfBr

[PATCH] D57345: Make clang/test/Index/pch-from-libclang.c pass in more places

sbc100 added a comment. Herald added a project: LLVM. I think this broke the BUILD_SHARED_LIBS build: https://reviews.llvm.org/D57556 Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57345/new/ https://reviews.llvm.org/D57345 __

[PATCH] D57521: [CMake] External compiler-rt-configure requires LLVMTestingSupport when including tests

beanz added a reviewer: phosek. beanz added a subscriber: phosek. beanz added a comment. Herald added a project: clang. @phosek, does LLVM’s runtime directory do this correctly? At some point we should try and deprecate and remove this option in favor of the LLVM directory because the LLVM appro

[PATCH] D50563: Fixed frontend clang tests in windows read-only container

justice_adams added a comment. @stella.stamenova @thakis Thanks for the feedback all, I will go ahead and make this change to the way I'm representing directories, open a new diff for review, and add you as a reviewer. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.or

[PATCH] D56611: [clangd] A code action to swap branches of an if statement

nridge added a comment. Herald added a project: LLVM. This commit is causing clangd to fail to link, with errors like this: tools/clang/tools/extra/clangd/refactor/tweaks/CMakeFiles/obj.clangDaemonTweaks.dir/SwapIfBranches.cpp.o:SwapIfBranches.cpp:function clang::RecursiveASTVisitor::Traverse

Re: r350920 - [Sema] Make canPassInRegisters return true if the CXXRecordDecl passed

+Hans for managing the 8.0 branch. On 1/31/19, 4:15 PM, "ahatan...@apple.com on behalf of Akira Hatanaka" wrote: Reverted patch in r352822. I’ll send a new patch later that disallows trivial_abi on classes without non-deleted copy or move constructors. > On Jan 31, 2019, at 3:52

Re: r351629 - Emit !callback metadata and introduce the callback attribute

Thanks for the clarifications. I'll fix all mentioned points asap. From: Eli Friedman Sent: Thursday, January 31, 2019 18:17 To: Doerfert, Johannes; Chandler Carruth; cfe-commits@lists.llvm.org Cc: Raja Venkateswaran Subject: RE: r351629 - Emit !callback metadata

RE: r351629 - Emit !callback metadata and introduce the callback attribute

(Comments inline.) > -Original Message- > From: cfe-commits On Behalf Of > Doerfert, Johannes Rudolf via cfe-commits > Sent: Tuesday, January 22, 2019 9:29 AM > To: Chandler Carruth > Cc: cfe-commits cfe > Subject: Re: r351629 - Emit !callback metadata and introduce the callback > attri

Re: r350920 - [Sema] Make canPassInRegisters return true if the CXXRecordDecl passed

Reverted patch in r352822. I’ll send a new patch later that disallows trivial_abi on classes without non-deleted copy or move constructors. > On Jan 31, 2019, at 3:52 PM, Richard Smith wrote: > > Given that there's uncertainty as to how to proceed and this patch > affects the ABI, I would prefe

r352822 - Revert "[Sema] Make canPassInRegisters return true if the CXXRecordDecl passed"

Author: ahatanak Date: Thu Jan 31 16:12:06 2019 New Revision: 352822 URL: http://llvm.org/viewvc/llvm-project?rev=352822&view=rev Log: Revert "[Sema] Make canPassInRegisters return true if the CXXRecordDecl passed" This reverts commit r350920 as it is not clear whether we should force a class to

[PATCH] D56900: [Fixed Point Arithmetic] Fixed Point and Integer Conversions

leonardchan marked an inline comment as done. leonardchan added inline comments. Comment at: clang/test/Frontend/fixed_point_conversions.c:437 + // DEFAULT-NEXT: [[RES:%[a-z0-9]+]] = trunc i39 [[SATMIN]] to i16 + // DEFAULT-NEXT: store i16 [[RES]], i16* %sat_sa, align 2 + -

[PATCH] D57553: [Fixed Point Arithmetic] Check against source value when converting during saturation

leonardchan created this revision. leonardchan added reviewers: rjmccall, ebevhan, bjope. leonardchan added a project: clang. When converting to a saturating fixed point type, compare against the source value instead of our container value. This allows us to not create a container, and save up t

Re: r350920 - [Sema] Make canPassInRegisters return true if the CXXRecordDecl passed

Given that there's uncertainty as to how to proceed and this patch affects the ABI, I would prefer that we revert it for trunk and 8.0. The suggested alternative of disallowing trivial_abi in the case where all copy/move constructors are deleted seems reasonable to me. On Thu, 31 Jan 2019 at 14:3

[PATCH] D57219: [Fixed Point Arithmetic] Fixed Point Comparisons

rjmccall added inline comments. Comment at: clang/lib/CodeGen/CGExprScalar.cpp:3444 case BO_NE: +return Builder.CreateICmpNE(FullLHS, FullRHS); + case BO_Mul: leonardchan wrote: > rjmccall wrote: > > Are padding bits guaranteed zero or unspecified? Or ar

r352818 - [sanitizer-coverage] prune trace-cmp instrumentation for CMP isntructions that feed into the backedge branch. Instrumenting these CMP instructions is almost always useless (and harmful) for

Author: kcc Date: Thu Jan 31 15:43:00 2019 New Revision: 352818 URL: http://llvm.org/viewvc/llvm-project?rev=352818&view=rev Log: [sanitizer-coverage] prune trace-cmp instrumentation for CMP isntructions that feed into the backedge branch. Instrumenting these CMP instructions is almost always us

Re: r352690 - [Sanitizers] UBSan unreachable incompatible with ASan in the presence of `noreturn` calls

Thank you, Eric, for the reproducer and instructions. Actionable and much appreciated.Can you test my updated patch on your build before I try again? updated_patch.patch Description: Binary data delta.diff Description: Binary data On Jan 31, 2019, at 6:19 AM, Eric Liu wrote

r352809 - Re-disable pch-from-libclang.c after 352803, some buildbots are still unhappy

Author: nico Date: Thu Jan 31 14:57:52 2019 New Revision: 352809 URL: http://llvm.org/viewvc/llvm-project?rev=352809&view=rev Log: Re-disable pch-from-libclang.c after 352803, some buildbots are still unhappy Modified: cfe/trunk/test/Index/pch-from-libclang.c Modified: cfe/trunk/test/Index/p

[PATCH] D57472: [AST] Extract ASTDumpTraverser class from ASTDumper

steveire updated this revision to Diff 184616. steveire added a comment. Update Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57472/new/ https://reviews.llvm.org/D57472 Files: include/clang/AST/ASTDumpTraverser.h lib/AST/ASTDumper.cpp Index: lib/AST/ASTDump

[PATCH] D57472: [AST] Extract ASTDumpTraverser class from ASTDumper

steveire marked an inline comment as done. steveire added inline comments. Comment at: include/clang/AST/ASTDumpTraverser.h:83 + + NodeVisitorType &getNodeVisitor() { return getDerived().doGetNodeVisitor(); } + Derived &getDerived() { return *static_cast(this); } --

[PATCH] D57219: [Fixed Point Arithmetic] Fixed Point Comparisons

leonardchan added inline comments. Comment at: clang/lib/CodeGen/CGExprScalar.cpp:3444 case BO_NE: +return Builder.CreateICmpNE(FullLHS, FullRHS); + case BO_Mul: rjmccall wrote: > Are padding bits guaranteed zero or unspecified? Or are we just not really

[PATCH] D57219: [Fixed Point Arithmetic] Fixed Point Comparisons

leonardchan updated this revision to Diff 184612. leonardchan marked 3 inline comments as done. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57219/new/ https://reviews.llvm.org/D57219 Files: clang/lib/AST/ExprConstant.cpp clang/lib/CodeGen/CGExprScalar.cpp

Re: r350920 - [Sema] Make canPassInRegisters return true if the CXXRecordDecl passed

Just wanted to point out that r350920 is on the 8.0 release branch as well. I don't know if there are any additional considerations there. On 1/31/19, 2:20 PM, "cfe-commits on behalf of John McCall via cfe-commits" wrote: On 31 Jan 2019, at 16:57, Akira Hatanaka wrote:

[PATCH] D57502: [ASTDump] Make template specialization tests more exact

This revision was automatically updated to reflect the committed changes. Closed by commit rL352804: [ASTDump] Make template specialization tests more exact (authored by steveire, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE LAST ACTION https://r

r352804 - [ASTDump] Make template specialization tests more exact

Author: steveire Date: Thu Jan 31 14:28:38 2019 New Revision: 352804 URL: http://llvm.org/viewvc/llvm-project?rev=352804&view=rev Log: [ASTDump] Make template specialization tests more exact Reviewers: aaron.ballman Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D57502

Re: r350920 - [Sema] Make canPassInRegisters return true if the CXXRecordDecl passed

On 31 Jan 2019, at 16:57, Akira Hatanaka wrote: Would it be better if we disallowed trivial_abi if the class’ copy and move destructors were all deleted (and revert r350920)? I think that would make it easier to reason about when you are allowed to use trivial_abi and what effect the attribu

[PATCH] D57345: Make clang/test/Index/pch-from-libclang.c pass in more places

This revision was automatically updated to reflect the committed changes. Closed by commit rL352803: Make clang/test/Index/pch-from-libclang.c pass in more places (authored by nico, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D57345

r352803 - Make clang/test/Index/pch-from-libclang.c pass in more places

Author: nico Date: Thu Jan 31 14:15:32 2019 New Revision: 352803 URL: http://llvm.org/viewvc/llvm-project?rev=352803&view=rev Log: Make clang/test/Index/pch-from-libclang.c pass in more places - fixes the test on macOS with LLVM_ENABLE_PIC=OFF - together with D57343, gets the test to pass on Wind

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

NoQ added a comment. P.S. I also support the idea to have a separate page per checker. It encourages people to write as much documentation as necessary and continously expand it. Otherwise there's a perception that all docs need to be roughly of the same size and shape in order to make the page

[PATCH] D57521: [CMake] External compiler-rt-configure requires LLVMTestingSupport when including tests

dberris accepted this revision. dberris added a comment. This revision is now accepted and ready to land. LGTM Thanks! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57521/new/ https://reviews.llvm.org/D57521 ___ cfe

[PATCH] D57527: Do not copy long double and 128-bit fp format from aux target for AMDGPU

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 rC352801: Do not copy long double and 128-bit fp format from aux target for AMDGPU (authored by yaxunl, committed by ). Cha

r352801 - Do not copy long double and 128-bit fp format from aux target for AMDGPU

Author: yaxunl Date: Thu Jan 31 13:57:51 2019 New Revision: 352801 URL: http://llvm.org/viewvc/llvm-project?rev=352801&view=rev Log: Do not copy long double and 128-bit fp format from aux target for AMDGPU rC352620 caused regressions because it copied floating point format from aux target. float

Re: r350920 - [Sema] Make canPassInRegisters return true if the CXXRecordDecl passed

Would it be better if we disallowed trivial_abi if the class’ copy and move destructors were all deleted (and revert r350920)? I think that would make it easier to reason about when you are allowed to use trivial_abi and what effect the attribute has (which is to override the trivialness for the

[PATCH] D57219: [Fixed Point Arithmetic] Fixed Point Comparisons

rjmccall added inline comments. Comment at: clang/lib/CodeGen/CGExprScalar.cpp:3444 case BO_NE: +return Builder.CreateICmpNE(FullLHS, FullRHS); + case BO_Mul: Are padding bits guaranteed zero or unspecified? Or are we just not really supporting padding

r352800 - Revert "[opaque pointer types] Add a FunctionCallee wrapper type, and use it."

Author: jyknight Date: Thu Jan 31 13:51:58 2019 New Revision: 352800 URL: http://llvm.org/viewvc/llvm-project?rev=352800&view=rev Log: Revert "[opaque pointer types] Add a FunctionCallee wrapper type, and use it." This reverts commit f47d6b38c7a61d50db4566b02719de05492dcef1 (r352791). Seems to r

[PATCH] D57532: [Index] Make sure c-index-test finds libc++ on Mac

arphaman requested changes to this revision. arphaman added a comment. This revision now requires changes to proceed. The real problem is that `clang_parseTranslationUnit2` is broken, so this just fixes one client, `c-index-test`. All other macOS clients that call that function will still be una

[PATCH] D57488: [CUDA] add support for the new kernel launch API in CUDA-9.2+.

This revision was automatically updated to reflect the committed changes. Closed by commit rC352799: [CUDA] add support for the new kernel launch API in CUDA-9.2+. (authored by tra, committed by ). Changed prior to commit: https://reviews.llvm.org/D57488?vs=184592&id=184598#toc Repository: r

r352799 - [CUDA] add support for the new kernel launch API in CUDA-9.2+.

Author: tra Date: Thu Jan 31 13:34:03 2019 New Revision: 352799 URL: http://llvm.org/viewvc/llvm-project?rev=352799&view=rev Log: [CUDA] add support for the new kernel launch API in CUDA-9.2+. Instead of calling CUDA runtime to arrange function arguments, the new API constructs arguments in a loc

[PATCH] D57487: [CUDA] Propagate detected version of CUDA to cc1

This revision was automatically updated to reflect the committed changes. Closed by commit rL352798: [CUDA] Propagate detected version of CUDA to cc1 (authored by tra, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D57487?vs=184416&id=

r352798 - [CUDA] Propagate detected version of CUDA to cc1

Author: tra Date: Thu Jan 31 13:32:24 2019 New Revision: 352798 URL: http://llvm.org/viewvc/llvm-project?rev=352798&view=rev Log: [CUDA] Propagate detected version of CUDA to cc1 ..and use it to control that parts of CUDA compilation that depend on the specific version of CUDA SDK. This patch ha

[PATCH] D57345: Make clang/test/Index/pch-from-libclang.c pass in more places

arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57345/new/ https://reviews.llvm.org/D57345 ___ cfe-commits mailing list cfe-commi

[PATCH] D57532: [Index] Make sure c-index-test finds libc++ on Mac

thakis added a comment. Looking at D54630 , it seems arphaman and dexonsmith are probalby good reviewers for this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57532/new/ https://reviews.llvm.org/D57532 ___

[PATCH] D56611: [clangd] A code action to swap branches of an if statement

This revision was automatically updated to reflect the committed changes. Closed by commit rL352796: [clangd] A code action to swap branches of an if statement (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D56611?vs=184588&id=184594#toc Repository:

[clang-tools-extra] r352796 - [clangd] A code action to swap branches of an if statement

Author: ibiryukov Date: Thu Jan 31 13:30:05 2019 New Revision: 352796 URL: http://llvm.org/viewvc/llvm-project?rev=352796&view=rev Log: [clangd] A code action to swap branches of an if statement Reviewers: sammccall Reviewed By: sammccall Subscribers: llvm-commits, mgorny, ioeric, MaskRay, jkor

[PATCH] D57345: Make clang/test/Index/pch-from-libclang.c pass in more places

thakis updated this revision to Diff 184593. thakis edited the summary of this revision. thakis added a comment. shared function CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57345/new/ https://reviews.llvm.org/D57345 Files: clang/include/clang/Driver/Driver.h clang/lib/Driver/Driv

[PATCH] D57488: [CUDA] add support for the new kernel launch API in CUDA-9.2+.

tra updated this revision to Diff 184592. tra added a comment. Updated ASTMatchers unit test. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57488/new/ https://reviews.llvm.org/D57488 Files: clang/include/clang/Basic/DiagnosticSemaKinds.td clang/include/clang/Sema/Sema.h clang/lib

[PATCH] D57540: [C++17] Don't crash while diagnosing different access specifier of a deduction guide.

Rakete created this revision. Rakete added a reviewer: rsmith. This fixes PR40552 by not showing the diagnostic that complains about different access specifiers, since the template does not have one. Repository: rC Clang https://reviews.llvm.org/D57540 Files: lib/Sema/SemaDeclCXX.

[PATCH] D56611: [clangd] A code action to swap branches of an if statement

ilya-biryukov updated this revision to Diff 184588. ilya-biryukov marked 8 inline comments as done. ilya-biryukov added a comment. Herald added a project: clang. Herald added a subscriber: llvm-commits. - Remove Dummy.cpp - Add halfOpenRangeTouches - Add a comment about file vs expansion locations

[PATCH] D56611: [clangd] A code action to swap branches of an if statement

ilya-biryukov added inline comments. Comment at: clang-tools-extra/clangd/SourceCode.h:63 +/// Turns a token range into a half-open range and checks its correctness. +/// The resulting range will have only valid source location on both sides, both sammccall wr

[PATCH] D55782: Fix isInSystemMacro to handle pasted token

rsmith accepted this revision. rsmith added inline comments. This revision is now accepted and ready to land. Comment at: include/clang/Basic/SourceManager.h:1466 +// its spelling is the scratch memory, so we take the parent context. +if(isWrittenInScratchSpace(getSpellin

Re: r352793 - [WebAssembly] Add bulk memory target feature

And another review that omitted lists. On Fri, Feb 1, 2019 at 12:02 AM Thomas Lively via cfe-commits wrote: > > Author: tlively > Date: Thu Jan 31 13:02:19 2019 > New Revision: 352793 > > URL: http://llvm.org/viewvc/llvm-project?rev=352793&view=rev > Log: > [WebAssembly] Add bulk memory target f

[PATCH] D56924: Handle ObjCCategoryDecl class extensions for print

arphaman requested changes to this revision. arphaman added a comment. This revision now requires changes to proceed. Do you know if this have an effect on the output of completion results or other tooling-based output? A couple of requests: - This behavior should be controlled by a printing po

[PATCH] D57527: Do not copy long double and 128-bit fp format from aux target for AMDGPU

yaxunl added a comment. In D57527#1379287 , @rjmccall wrote: > Explanatory comment, please. Otherwise LGTM. will do when committing. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57527/new/ https://reviews.llvm.org/D57527 _

r352793 - [WebAssembly] Add bulk memory target feature

Author: tlively Date: Thu Jan 31 13:02:19 2019 New Revision: 352793 URL: http://llvm.org/viewvc/llvm-project?rev=352793&view=rev Log: [WebAssembly] Add bulk memory target feature Summary: Also clean up some preexisting target feature code. Reviewers: aheejin Subscribers: dschuff, sbc100, jgrave

[PATCH] D57345: Make clang/test/Index/pch-from-libclang.c pass in more places

arphaman added a comment. Yes, that should work. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57345/new/ https://reviews.llvm.org/D57345 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinf

[PATCH] D57438: [Sema][ObjC] Allow declaring ObjC pointer members in C++ unions under ARC

rjmccall added inline comments. Comment at: lib/Sema/SemaDeclCXX.cpp:7033 + // member of an ObjC pointer type, except when it has an in-class initializer, + // it doesn't make the defaulted default constructor defined as deleted. + if (!FieldType.hasNonTrivialObjCLifetime() |

[PATCH] D57315: [opaque pointer types] Add a FunctionCallee wrapper type, and use it.

This revision was automatically updated to reflect the committed changes. Closed by commit rC352791: [opaque pointer types] Add a FunctionCallee wrapper type, and use it. (authored by jyknight, committed by ). Herald added a subscriber: cfe-commits. Changed prior to commit: https://reviews.llvm

[PATCH] D57527: Do not copy long double and 128-bit fp format from aux target for AMDGPU

rjmccall added a comment. Explanatory comment, please. Otherwise LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57527/new/ https://reviews.llvm.org/D57527 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/

r352791 - [opaque pointer types] Add a FunctionCallee wrapper type, and use it.

Author: jyknight Date: Thu Jan 31 12:35:56 2019 New Revision: 352791 URL: http://llvm.org/viewvc/llvm-project?rev=352791&view=rev Log: [opaque pointer types] Add a FunctionCallee wrapper type, and use it. The FunctionCallee type is effectively a {FunctionType*,Value*} pair, and is a useful conven

[PATCH] D54429: [analyzer] Creating standard Sphinx documentation

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. I think let's get this up and running and slowly redirect links from the front page to the new docs until only the front page is left? Comment at: docs/analyzer/checkers.rst:36-37

[PATCH] D57527: Do not copy long double and 128-bit fp format from aux target for AMDGPU

yaxunl added a comment. In D57527#1379159 , @rjmccall wrote: > In D57527#1379088 , @yaxunl wrote: > > > In D57527#1379065 , @rjmccall > > wrote: > > > > > Okay, so you silen

[PATCH] D57527: Do not copy long double and 128-bit fp format from aux target for AMDGPU

yaxunl updated this revision to Diff 184569. yaxunl retitled this revision from "Do not copy floating pointer format from aux target" to "Do not copy long double and 128-bit fp format from aux target for AMDGPU". yaxunl edited the summary of this revision. yaxunl added a comment. Herald added sub

r352788 - [CodeComplete] Propagate preferred types through parser in more cases

Author: ibiryukov Date: Thu Jan 31 12:20:32 2019 New Revision: 352788 URL: http://llvm.org/viewvc/llvm-project?rev=352788&view=rev Log: [CodeComplete] Propagate preferred types through parser in more cases Preferred types are used by code completion for ranking. This commit considerably increases

[PATCH] D56723: [CodeComplete] Propagate preferred types through parser in more cases

This revision was automatically updated to reflect the committed changes. Closed by commit rC352788: [CodeComplete] Propagate preferred types through parser in more cases (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D56723?vs=184038&id=184568#toc Re

[PATCH] D57472: [AST] Extract ASTDumpTraverser class from ASTDumper

aaron.ballman added inline comments. Comment at: include/clang/AST/ASTDumpTraverser.h:31 + +ASTDumpTraverser traverses the Clang AST for dumping purposes + Missing full stop at the end of the sentence. Comment at: include/clang/AST/ASTDumpTrave

[PATCH] D57345: Make clang/test/Index/pch-from-libclang.c pass in more places

thakis added a comment. Could it be somewhere in lib/Driver? libclang depends on lib/Frontend which depends on lib/Driver -- is there any reason libclang can't depend on lib/Driver? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57345/new/ https://reviews.llvm.org/D57345 ___

[PATCH] D57532: [Index] Make sure c-index-test finds libc++ on Mac

ilya-biryukov added a comment. I'm not sure who the owners of c-index-test are, so adding people who touched the file recently or might be interested in this landing. Suggestions for other reviewers are **very** welcome. Still struggling to make one test pass: `Index/index-file.cu` fails with:

[PATCH] D57532: [Index] Make sure c-index-test finds libc++ on Mac

ilya-biryukov created this revision. ilya-biryukov added reviewers: thakis, ioeric. Herald added a reviewer: EricWF. Herald added subscribers: llvm-commits, arphaman. Herald added a project: clang. Specifically, the version of libc++ that lives alongside the c-index-test binary. After r348365 driv

[PATCH] D57527: Do not copy floating pointer format from aux target

rjmccall added a comment. In D57527#1379088 , @yaxunl wrote: > In D57527#1379065 , @rjmccall wrote: > > > Okay, so you silently have an incompatible ABI for anything in the system > > headers that mentions `long do

[PATCH] D57207: [clang-tidy] Make google-objc-function-naming ignore implicit functions 🙈

stephanemoore updated this revision to Diff 184553. stephanemoore marked an inline comment as done. stephanemoore added a comment. Add comment explaining that builtin function call generates implicit function declaration and update calling function name for improved clarity. Repository: rCTE

[PATCH] D57207: [clang-tidy] Make google-objc-function-naming ignore implicit functions 🙈

stephanemoore marked 3 inline comments as done. stephanemoore added a comment. Thanks for the review! Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57207/new/ https://reviews.llvm.org/D57207 ___ cfe-com

[PATCH] D57527: Do not copy floating pointer format from aux target

yaxunl added a comment. In D57527#1379065 , @rjmccall wrote: > Okay, so you silently have an incompatible ABI for anything in the system > headers that mentions `long double`. Do you have any plans to address or > work around that, or is the hope that i

[PATCH] D57207: [clang-tidy] Make google-objc-function-naming ignore implicit functions 🙈

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM with a commenting request. Comment at: test/clang-tidy/google-objc-function-naming.m:5-8 +static void TestImplicitlyDefinedFunction(int a) { + printf("%d"

[PATCH] D45978: dllexport const variables must have external linkage.

aaron.ballman added a reviewer: majnemer. aaron.ballman added a subscriber: majnemer. aaron.ballman added inline comments. Comment at: test/Sema/dllexport.c:70 +// const variables +__declspec(dllexport) int const x = 3; zahiraam wrote: > aaron.ballman wrote: >

[PATCH] D57527: Do not copy floating pointer format from aux target

rjmccall added a comment. Okay, so you silently have an incompatible ABI for anything in the system headers that mentions `long double`. Do you have any plans to address or work around that, or is the hope that it just doesn't matter? I feel like this should be a special case for AMDGPU rather

  1   2   >