[PATCH] D73615: [clangd] Get rid of delayed template parsing

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts :

[PATCH] D73610: [clangd] Replace raw lexer code with token buffer in prepare rename.

hokein updated this revision to Diff 241096. hokein added a comment. revert an accident change. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73610/new/ https://reviews.llvm.org/D73610 Files: clang-tools-extra/clangd/ClangdServer.cpp

[PATCH] D73619: [clangd] Remove the temporary alias for clangd::DiagnosticConsumer.

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Thanks for the cleanup! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73619/new/ https://reviews.llvm.org/D73619

[PATCH] D72153: [libTooling] Add function to determine associated text of a declaration.

ymandel updated this revision to Diff 241127. ymandel marked 12 inline comments as done. ymandel added a comment. tweaks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72153/new/ https://reviews.llvm.org/D72153 Files:

[PATCH] D73543: [clang] Add support for __builtin_memcpy_inline

gchatelet added inline comments. Comment at: clang/docs/LanguageExtensions.rst:2259 +``__builtin_memcpy_inline(dst, src, size)`` is identical to +``__builtin_memcpy(dst, src, size)`` expect that the generated code is +guaranteed not to call any external functions. See [LLVM IR

[PATCH] D72326: [clang-format] Rebased on master: Add option to specify explicit config file

tnorth updated this revision to Diff 241155. tnorth added a comment. - Add release notes - Update ClangFormat.rst and ClangFormatStyleOption.rst Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72326/new/ https://reviews.llvm.org/D72326 Files:

[PATCH] D72326: [clang-format] Rebased on master: Add option to specify explicit config file

tnorth added a comment. In D72326#1845342 , @MyDeveloperDay wrote: > Nit: please add a release note and regenerate the ClangFormatStyleOptions.rst > (if there are any changes because you modified Format.h). Hmm, I tried to run

[PATCH] D73628: [clangd] Log directory when a CDB is loaded

kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Fixes https://github.com/clangd/clangd/issues/268 Repository: rG LLVM Github Monorepo

[clang-tools-extra] 55b0e9c - [clangd][Hover] Make tests hermetic by setting target triplet

Author: Kadir Cetinkaya Date: 2020-01-29T12:12:45+01:00 New Revision: 55b0e9c9d5de7c5d70552ac9ca9ffc14097e983b URL: https://github.com/llvm/llvm-project/commit/55b0e9c9d5de7c5d70552ac9ca9ffc14097e983b DIFF:

[PATCH] D73613: [clangd][Hover] Make tests hermetic by setting target triplet

sammccall added inline comments. Comment at: clang-tools-extra/clangd/unittests/HoverTests.cpp:586 +// fixed one to make sure tests passes on different platform. +TU.ExtraArgs.push_back("--target=x86_64-pc-linux-gnu"); auto AST = TU.build();

[clang-tools-extra] 7830c2d - [clangd] Get rid of delayed template parsing

Author: Kadir Cetinkaya Date: 2020-01-29T12:12:45+01:00 New Revision: 7830c2d44f531bbe09f997436b6608a140db46fb URL: https://github.com/llvm/llvm-project/commit/7830c2d44f531bbe09f997436b6608a140db46fb DIFF:

[PATCH] D73619: [clangd] Remove the temporary alias for clangd::DiagnosticConsumer.

hokein updated this revision to Diff 241122. hokein added a comment. remove the deprecated contructor as well. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73619/new/ https://reviews.llvm.org/D73619 Files:

[clang] 2939fc1 - [AArch64] Add IR intrinsics for sq(r)dmulh_lane(q)

Author: Sanne Wouda Date: 2020-01-29T13:25:23Z New Revision: 2939fc13c8f6a5dbd1be77c1d19dc2720253b8c5 URL: https://github.com/llvm/llvm-project/commit/2939fc13c8f6a5dbd1be77c1d19dc2720253b8c5 DIFF: https://github.com/llvm/llvm-project/commit/2939fc13c8f6a5dbd1be77c1d19dc2720253b8c5.diff LOG:

[PATCH] D72222: [Driver][CodeGen] Add -fpatchable-function-entry=N[,0]

mrutland added a comment. In D7#1839207 , @MaskRay wrote: > When -mbranch-protection=bti is used, due to > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92424 (existing GCC releases > have the issue), Linux/arch/arm64 has to make more checks to

[PATCH] D73242: [WPD/LowerTypeTests] Delay lowering/removal of type tests until after ICP

evgeny777 added a comment. > This is an enabler for upcoming enhancements to indirect call promotion, for > example streamlined promotion guard sequences that compare against vtable > address instead of the target function Can you please describe the whole approach in more detail? At the

[PATCH] D73610: [clangd] Replace raw lexer code with token buffer in prepare rename.

hokein added inline comments. Comment at: clang-tools-extra/clangd/ClangdServer.cpp:322 +if (!TouchingIdentifier) + return CB(llvm::None); // no rename on non-identifiers. + sammccall wrote: > Agree that this behavior is OK per the spec, but do we

[clang-tools-extra] fce8983 - [clangd] Remove the temporary alias for clangd::DiagnosticConsumer.

Author: Haojian Wu Date: 2020-01-29T14:50:58+01:00 New Revision: fce8983a3c03b41c3ba4bdaef72e64e29ff9ecc0 URL: https://github.com/llvm/llvm-project/commit/fce8983a3c03b41c3ba4bdaef72e64e29ff9ecc0 DIFF: https://github.com/llvm/llvm-project/commit/fce8983a3c03b41c3ba4bdaef72e64e29ff9ecc0.diff

[PATCH] D73285: [OpenMP][OMPIRBuilder][BugFix] Handle Unreachable Finalization blocks in `parallel` generation

fghanim marked an inline comment as done. fghanim added inline comments. Comment at: llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp:661 +AfterIP = InsertPointTy(ForkBB, ForkBB->end()); + } jdoerfert wrote: > fghanim wrote: > > jdoerfert wrote: > > > Why do we

[clang-tools-extra] bcb3e42 - [clangd] Go-to-definition on 'override' jumps to overridden method(s)

Author: Sam McCall Date: 2020-01-29T12:43:52+01:00 New Revision: bcb3e42fdfb30f516fefd56609a969059b60a982 URL: https://github.com/llvm/llvm-project/commit/bcb3e42fdfb30f516fefd56609a969059b60a982 DIFF: https://github.com/llvm/llvm-project/commit/bcb3e42fdfb30f516fefd56609a969059b60a982.diff

[PATCH] D73354: clang-format: insert trailing commas into containers.

mprobst marked 4 inline comments as done. mprobst added inline comments. Comment at: clang/include/clang/Format/Format.h:42 + Unsuitable, + BinBackTrailingCommaConflict +}; sammccall wrote: > Back->Pack? That's what you get when you fix my auto-complete:

[PATCH] D73617: [clangd] Don't mmap source files on all platforms --> don't crash on git checkout

sammccall added a comment. Also I'd like to put this onto the 10 release branch because this is a big crasher, fix seems safe, we've run this code on windows for a while. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73617/new/

[clang] d5dfd13 - Add TagDecl AST matcher

Author: Karasev Nikita Date: 2020-01-29T07:58:31-05:00 New Revision: d5dfd1350efb80f9674db322999dd883fb36a6ad URL: https://github.com/llvm/llvm-project/commit/d5dfd1350efb80f9674db322999dd883fb36a6ad DIFF:

[clang] 4ec2a26 - Fix clang test build

Author: Sanne Wouda Date: 2020-01-29T13:03:27Z New Revision: 4ec2a267321124a7fe8efe794ce40da67ce1d6bd URL: https://github.com/llvm/llvm-project/commit/4ec2a267321124a7fe8efe794ce40da67ce1d6bd DIFF: https://github.com/llvm/llvm-project/commit/4ec2a267321124a7fe8efe794ce40da67ce1d6bd.diff LOG:

[PATCH] D73548: [clang-tidy] Added option for disabling const qualifiers in readability-qualified-auto

aaron.ballman added inline comments. Comment at: clang-tools-extra/docs/clang-tidy/checks/readability-qualified-auto.rst:41 + auto *const Bar = cast(Baz2); + auto *volatile FooBar = cast(Baz3); + njames93 wrote: > aaron.ballman wrote: > > njames93 wrote: >

[PATCH] D73464: [clang] Add TagDecl AST matcher

aaron.ballman closed this revision. aaron.ballman added a comment. I've committed on your behalf in d5dfd1350efb80f9674db322999dd883fb36a6ad , thank you for the patch! Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D73562: [ASTMatchers] Add hasPlacementArg and hasAnyPlacementArg traversal matcher for CXXNewExpr

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73562/new/ https://reviews.llvm.org/D73562

[clang-tools-extra] 01213f9 - [clang-tidy] Initialize token before handing it to the lexer

Author: Benjamin Kramer Date: 2020-01-29T16:48:57+01:00 New Revision: 01213f90700dbb98a0dbcc01da8fdb89f6db5617 URL: https://github.com/llvm/llvm-project/commit/01213f90700dbb98a0dbcc01da8fdb89f6db5617 DIFF:

[PATCH] D73628: [clangd] Log directory when a CDB is loaded

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/GlobalCompilationDatabase.cpp:122 +if (Entry.CDB) + vlog("Loaded CDB from {0}", Dir); } consider

[clang-tools-extra] 6f69527 - [clangd] add CODE_OWNERS

Author: Sam McCall Date: 2020-01-29T12:43:19+01:00 New Revision: 6f6952780ba92782c38b37b7bf65a079c1d1215f URL: https://github.com/llvm/llvm-project/commit/6f6952780ba92782c38b37b7bf65a079c1d1215f DIFF: https://github.com/llvm/llvm-project/commit/6f6952780ba92782c38b37b7bf65a079c1d1215f.diff

[PATCH] D73537: [clangd] add CODE_OWNERS

This revision was automatically updated to reflect the committed changes. Closed by commit rG6f6952780ba9: [clangd] add CODE_OWNERS (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73537/new/ https://reviews.llvm.org/D73537

[clang-tools-extra] e864f93 - [clangd] Replace raw lexer code with token buffer in prepare rename.

Author: Haojian Wu Date: 2020-01-29T12:57:18+01:00 New Revision: e864f937669c996b4dc15db7d0ebe4073527c165 URL: https://github.com/llvm/llvm-project/commit/e864f937669c996b4dc15db7d0ebe4073527c165 DIFF: https://github.com/llvm/llvm-project/commit/e864f937669c996b4dc15db7d0ebe4073527c165.diff

[clang-tools-extra] 17fadef - [clangd][vscode] Update lsp dependencies to pickup the progress support in LSP 3.15

Author: Haojian Wu Date: 2020-01-29T12:58:53+01:00 New Revision: 17fadeffcce97d79f5f132dacb143364622b6428 URL: https://github.com/llvm/llvm-project/commit/17fadeffcce97d79f5f132dacb143364622b6428 DIFF: https://github.com/llvm/llvm-project/commit/17fadeffcce97d79f5f132dacb143364622b6428.diff

[PATCH] D73610: [clangd] Replace raw lexer code with token buffer in prepare rename.

This revision was automatically updated to reflect the committed changes. Closed by commit rGe864f937669c: [clangd] Replace raw lexer code with token buffer in prepare rename. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D73610: [clangd] Replace raw lexer code with token buffer in prepare rename.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62257 tests passed, 0 failed and 827 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D73612: [clangd] Update lsp dependencies to pickup the progress support in LSP 3.15

This revision was automatically updated to reflect the committed changes. Closed by commit rG17fadeffcce9: [clangd][vscode] Update lsp dependencies to pickup the progress support in LSP… (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D73354: clang-format: insert trailing commas into containers.

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. (Maybe we should have a basic test for C++, but I'm not sure how this usually goes) BTW it occurs to me that turning this on by default may be a surprising breakage for: - people who

[PATCH] D73619: [clangd] Remove the temporary alias for clangd::DiagnosticConsumer.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62283 tests passed, 0 failed and 827 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D71469: [AArch64] Add IR intrinsics for sq(r)dmulh_lane(q)

This revision was automatically updated to reflect the committed changes. Closed by commit rG2939fc13c8f6: [AArch64] Add IR intrinsics for sq(r)dmulh_lane(q) (authored by sanwou01). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71469/new/

[PATCH] D72829: Implement -fsemantic-interposition

serge-sans-paille updated this revision to Diff 241152. serge-sans-paille added a comment. Update user-level documentation. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72829/new/ https://reviews.llvm.org/D72829 Files:

[PATCH] D73612: [clangd] Update lsp dependencies to pickup the progress support in LSP 3.15

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. LG. (The internal google mirror of VSCode is stuck at 1.38, but this should just fail to update the extension I think) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D73610: [clangd] Replace raw lexer code with token buffer in prepare rename.

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/ClangdServer.cpp:322 +if (!TouchingIdentifier) + return CB(llvm::None); // no rename on non-identifiers. +

[PATCH] D73617: [clangd] Don't mmap source files on all platforms --> don't crash on git checkout

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62284 tests passed, 0 failed and 831 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D73612: [clangd] Update lsp dependencies to pickup the progress support in LSP 3.15

hokein updated this revision to Diff 241097. hokein added a comment. update the required minimal vscode version. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73612/new/ https://reviews.llvm.org/D73612 Files:

[PATCH] D73360: [OpenCL] Restrict address space conversions in nested pointers

jeroen.dobbelaere accepted this revision. jeroen.dobbelaere added a comment. This patch looks good to me. I do agree with John that it would be good to add the warning to a warning group. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73360/new/ https://reviews.llvm.org/D73360

[PATCH] D73570: [FPEnv][X86] Platform-specific builtin constrained FP enablement

kpn marked 2 inline comments as done. kpn added inline comments. Comment at: clang/test/CodeGen/fma-builtins-constrained.c:4 +// RUN: %clang_cc1 -ffreestanding %s -triple=x86_64-apple-darwin -target-feature +fma -S -o - | FileCheck --check-prefix=COMMON

[PATCH] D72829: Implement -fsemantic-interposition

sfertile added a comment. Aewsome, thanks for implementing this. I was on vacation for a bit and somehow missed this review in my queue when I cam back but having a look at it now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72829/new/

[PATCH] D73624: [clangd][vscode] Get rid of the deprecated vscode module in the extension.

hokein created this revision. hokein added a reviewer: sammccall. Herald added subscribers: usaxena95, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov, dschuff. Herald added a project: clang. hokein updated this revision to Diff 241144. hokein added a comment. add trailing blank line. The

[PATCH] D73462: [dwarf-5] Support DebugInfo for Defaulted parameters for C++ templates

awpandey updated this revision to Diff 241141. awpandey added a comment. Thanks @dblaikie for figuring out the missing feature. I have added support for the non-type template parameter too. Please let me know if further modifications are required. I will happily address them. CHANGES SINCE

[PATCH] D73624: [clangd][vscode] Get rid of the deprecated vscode module in the extension.

hokein updated this revision to Diff 241144. hokein added a comment. add trailing blank line. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73624/new/ https://reviews.llvm.org/D73624 Files:

[PATCH] D73543: [clang] Add support for __builtin_memcpy_inline

gchatelet updated this revision to Diff 241158. gchatelet marked 4 inline comments as done. gchatelet added a comment. - Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73543/new/ https://reviews.llvm.org/D73543 Files:

[PATCH] D73629: [analyzer] vfork checker: allow execve after vfork

janvcelak created this revision. janvcelak added a reviewer: dcoughlin. Herald added subscribers: cfe-commits, Charusso, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Herald added a project: clang. `execve` is missing in the list of functions

[PATCH] D73617: [clangd] Don't mmap source files on all platforms --> don't crash on git checkout

sammccall created this revision. Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. sammccall added reviewers: ilya-biryukov, kadircet. sammccall edited the summary of this revision. Previously we mmapped on unix

[PATCH] D72153: [libTooling] Add function to determine associated text of a declaration.

ymandel updated this revision to Diff 241124. ymandel marked 4 inline comments as done. ymandel added a comment. addressed comments, with significant reworking of tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72153/new/

[PATCH] D73619: [clangd] Remove the temporary alias for clangd::DiagnosticConsumer.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62283 tests passed, 0 failed and 827 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D73619: [clangd] Remove the temporary alias for clangd::DiagnosticConsumer.

This revision was automatically updated to reflect the committed changes. Closed by commit rGfce8983a3c03: [clangd] Remove the temporary alias for clangd::DiagnosticConsumer. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D72153: [libTooling] Add function to determine associated text of a declaration.

ymandel marked an inline comment as done. ymandel added a comment. Thank you for the detailed review. I've significantly expanded and refactored the tests. I also lifted `validateEditRange` into its own function and added corresponding tests. Comment at:

[PATCH] D73613: [clangd][Hover] Make tests hermetic by setting target triplet

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62269 tests passed, 0 failed and 827 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D73613: [clangd][Hover] Make tests hermetic by setting target triplet

This revision was automatically updated to reflect the committed changes. Closed by commit rG55b0e9c9d5de: [clangd][Hover] Make tests hermetic by setting target triplet (authored by kadircet). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73613/new/

[PATCH] D73615: [clangd] Get rid of delayed template parsing

This revision was automatically updated to reflect the committed changes. Closed by commit rG7830c2d44f53: [clangd] Get rid of delayed template parsing (authored by kadircet). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73615/new/

[PATCH] D73354: clang-format: insert trailing commas into containers.

mprobst marked 2 inline comments as done. mprobst added inline comments. Comment at: clang/lib/Format/Format.cpp:2531 +}); + auto Env = MyDeveloperDay wrote: > Ok, this comment is more a discussion point rather than a review comment. Was > there a reason

[PATCH] D73367: [clangd] Go-to-definition on 'override' jumps to overridden method(s)

This revision was automatically updated to reflect the committed changes. Closed by commit rGbcb3e42fdfb3: [clangd] Go-to-definition on override jumps to overridden method(s) (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D73619: [clangd] Remove the temporary alias for clangd::DiagnosticConsumer.

hokein added a comment. We have updated our internal code to use the new name. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73619/new/ https://reviews.llvm.org/D73619 ___ cfe-commits mailing list

[PATCH] D73619: [clangd] Remove the temporary alias for clangd::DiagnosticConsumer.

hokein created this revision. hokein added a reviewer: sammccall. Herald added subscribers: usaxena95, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. hokein added a comment. We have updated our internal code to use the new name. Repository: rG LLVM Github

[PATCH] D73493: [AArch64][SVE] Add SVE2 intrinsics for uniform DSP operations

This revision was automatically updated to reflect the committed changes. Closed by commit rG3cf80822a906: [AArch64][SVE] Add SVE2 intrinsics for uniform DSP operations (authored by kmclaughlin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D73610: [clangd] Replace raw lexer code with token buffer in prepare rename.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62257 tests passed, 0 failed and 827 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D73612: [clangd] Update lsp dependencies to pickup the progress support in LSP 3.15

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62283 tests passed, 0 failed and 827 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[clang] a324fcf - clang-format: insert trailing commas into containers.

Author: Martin Probst Date: 2020-01-29T13:23:54+01:00 New Revision: a324fcf1ae62d065b957e66a9d2f5c18b6259d27 URL: https://github.com/llvm/llvm-project/commit/a324fcf1ae62d065b957e66a9d2f5c18b6259d27 DIFF: https://github.com/llvm/llvm-project/commit/a324fcf1ae62d065b957e66a9d2f5c18b6259d27.diff

[PATCH] D73610: [clangd] Replace raw lexer code with token buffer in prepare rename.

hokein updated this revision to Diff 241095. hokein marked 4 inline comments as done. hokein added a comment. address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73610/new/ https://reviews.llvm.org/D73610 Files:

[PATCH] D73624: [clangd][vscode] Get rid of the deprecated vscode module in the extension.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62287 tests passed, 0 failed and 831 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D72829: Implement -fsemantic-interposition

serge-sans-paille added a comment. > As this is user-facing documentation I feel like there should be a slightly > longer explaning what the option does. done! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72829/new/

[PATCH] D73624: [clangd][vscode] Get rid of the deprecated vscode module in the extension.

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62285 tests passed, 2 failed and 831 were skipped. failed: libc++.std/thread/thread_mutex/thread_mutex_requirements/thread_mutex_requirements_mutex/thread_mutex_recursive/lock.pass.cpp failed:

[PATCH] D73638: [AST] Move dependence computations into a separate file

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts :

[PATCH] D73651: [OpenCL][CUDA][HIP] Add norecurse

yaxunl created this revision. yaxunl added reviewers: Anastasia, tra. norecurse function attr indicates the function is not called recursively directly or indirectly. Add norecurse to OpenCL functions and CUDA/HIP kernels. Although there is LLVM pass adding norecurse to functions, it only

[PATCH] D72222: [Driver][CodeGen] Add -fpatchable-function-entry=N[,0]

mrutland added a comment. In D7#1846796 , @mrutland wrote: > In D7#1839207 , @MaskRay wrote: > > > I shall also mention that we are essentially making decisions for x86 > > people's endbr32/endbr64. I hope

[clang] 6e82d0d - [Clang][Bundler] Add 'exclude' flag to target objects sections

Author: Sergey Dmitriev Date: 2020-01-29T09:00:45-08:00 New Revision: 6e82d0dfd8dfaebc3985e73740a020b273a2dd31 URL: https://github.com/llvm/llvm-project/commit/6e82d0dfd8dfaebc3985e73740a020b273a2dd31 DIFF:

[PATCH] D57054: [MachineOutliner][ARM][RFC] Add Machine Outliner support for ARM

yroux updated this revision to Diff 241188. yroux added a comment. Here is a new version, which disables ARM Low Overhead Loops pass when the Machine Outliner is enabled. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57054/new/

[PATCH] D73617: [clangd] Don't mmap source files on all platforms --> don't crash on git checkout

ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73617/new/ https://reviews.llvm.org/D73617

[PATCH] D73649: [CodeComplete] Member completion for concept-constrained types.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62282 tests passed, 0 failed and 827 were skipped. {icon times-circle color=red} clang-tidy: fail. clang-tidy found 0 errors and 7 warnings

[PATCH] D73629: [analyzer] vfork checker: allow execve after vfork

xazax.hun added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/VforkChecker.cpp:109 "execvpe", + "execve", nullptr Well, this is not the case now, but I wonder if it would also make sense to sort this list alphabetically.

[PATCH] D73437: [mlir][spirv] Convert linalg.generic for reduction to SPIR-V ops

mravishankar accepted this revision. mravishankar marked 2 inline comments as done. mravishankar added inline comments. Comment at: mlir/lib/Conversion/LinalgToSPIRV/LinalgToSPIRV.cpp:73 + + using mlir::matchers::m_Val; + auto a = m_Val(block.getArgument(0));

[PATCH] D73636: [AArch64][SVE] SVE2 intrinsics for complex integer arithmetic

kmclaughlin created this revision. kmclaughlin added reviewers: sdesmalen, dancgr, efriedma, cameron.mcinally, c-rhodes. Herald added subscribers: psnobl, rkruppe, hiraditya, kristof.beyls, tschuett. Herald added a reviewer: rengolin. Herald added a project: LLVM. Adds the following SVE2

[PATCH] D73637: Fix handling of OO_Spaceship in DecodeOperatorCall

sberg created this revision. sberg added reviewers: rsmith, EricWF. sberg added a project: clang. Herald added a subscriber: cfe-commits. This seems to be a leftover from d30b23d6a54b3f0883914f3c2c2318a78edcbe67 "[c++2a] P0515R3: Support for overloaded operator<=>." (The corresponding

[PATCH] D73642: [Clang][Bundler] Reduce fat object size

sdmitriev created this revision. sdmitriev added a reviewer: ABataev. Herald added a reviewer: alexshap. Herald added a reviewer: jdoerfert. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fat object size has significantly increased after D65819

[PATCH] D73644: [Mips] Add intrinsics for 4-byte and 8-byte MSA loads/stores.

mbrkusanin added a comment. A few notes/questions: 1. Generated code was tested with Qemu: - For mips32r5 Qemu provides p5600 - For mips64r6 Qemu provides i6400 - For mips64r5 there is no cpu on Qemu with MSA and it appears that there won't be any hardware with Mips64r5 and MSA. - For

[PATCH] D72829: Implement -fsemantic-interposition

serge-sans-paille updated this revision to Diff 241254. serge-sans-paille added a comment. take @sfertile review into account. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72829/new/ https://reviews.llvm.org/D72829 Files:

[PATCH] D72304: [OpenMP][OMPIRBuilder] Add Directives (master and critical) to OMPBuilder.

fghanim added a comment. Everything is fine! I just cloned llvm from git hub, added this revision with `arc patch`. then: `cmake -G"Unix Makefiles" -DLLVM_ENABLE_PROJECTS="clang;openmp" -DCMAKE_BUILD_TYPE=Release -DLLVM_OPTIMIZED_TABLEGEN=1 -DLLVM_USE_LINKER=gold ../llvm-project/llvm` `make` ,

[PATCH] D73638: [AST] Move dependence computations into a separate file

ilya-biryukov added a comment. @rsmith this does not pass through testing, I've messed up somewhere while moving the code. I'll find what went wrong and fix it tomorrow. Please tell if the approach itself LG. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D72829: Implement -fsemantic-interposition

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62300 tests passed, 0 failed and 837 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D73300: [clang-tidy] Add library for clang-tidy main function

DmitryPolukhin added a comment. @thakis I'm sorry sorry if it was not clear. Please let me know if you still prefer to have separate directory for clangTidyMain to have only one target per CMakeLists.txt. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 8093d37 - Fix switch covers all cases static analyzer warning. NFCI.

Author: Simon Pilgrim Date: 2020-01-29T17:26:22Z New Revision: 8093d37ed2548ff5c8d6d7c51bd0431afbbf4209 URL: https://github.com/llvm/llvm-project/commit/8093d37ed2548ff5c8d6d7c51bd0431afbbf4209 DIFF: https://github.com/llvm/llvm-project/commit/8093d37ed2548ff5c8d6d7c51bd0431afbbf4209.diff

[PATCH] D73629: [analyzer] vfork checker: allow execve after vfork

Charusso added a reviewer: NoQ. Charusso added a comment. Hey, thanks! The patch looks great, but please note that we do the reviews with context using `git diff -U99` or uploading with `arc` (https://secure.phabricator.com/book/phabricator/article/arcanist/). Repository: rG LLVM Github

[PATCH] D73628: [clangd] Log directory when a CDB is loaded

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62284 tests passed, 0 failed and 831 were skipped. {icon check-circle color=green} clang-tidy: pass. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D72829: Implement -fsemantic-interposition

sfertile added a comment. In D72829#1846353 , @arichardson wrote: > As this is user-facing documentation I feel like there should be a slightly > longer explaning what the option does. +1 on this, otherwise LGTM. Thanks for implementing this!

[PATCH] D73408: [Clang][Bundler] Add 'exclude' flag to target objects sections

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

[PATCH] D73408: [Clang][Bundler] Add 'exclude' flag to target objects sections

This revision was automatically updated to reflect the committed changes. Closed by commit rG6e82d0dfd8df: [Clang][Bundler] Add exclude flag to target objects sections (authored by sdmitriev). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73408/new/

[PATCH] D73644: [Mips] Add intrinsics for 4-byte and 8-byte MSA loads/stores.

mbrkusanin created this revision. mbrkusanin added reviewers: atanasyan, petarj, sdardis, mstojanovic. mbrkusanin added projects: LLVM, clang. Herald added subscribers: cfe-commits, jrtc27, hiraditya, arichardson. New intrinisics are implemented for when we need to port SIMD code from other

[PATCH] D73651: [OpenCL][CUDA][HIP] Add norecurse

tra accepted this revision. tra added a comment. This revision is now accepted and ready to land. LGTM for CUDA. Comment at: clang/lib/CodeGen/CodeGenFunction.cpp:918 + // + // ToDo: clang does not support CUDA/HIP dynamic parallelism, therefore + // CUDA/HIP kernel can be

[PATCH] D73642: [Clang][Bundler] Reduce fat object size

ABataev added inline comments. Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:493-518 +// Temporary files that need to be removed. +struct TempFileList : public SmallVector, 2u> { + ~TempFileList() { +for (const auto : *this) +

[PATCH] D73629: [analyzer] vfork checker: allow execve after vfork

NoQ added a comment. Yup, thanks, nice catch! We should add a test for this (cf. `test/Analysis/vfork.c`). Also do you have any immediate opinions on https://bugs.llvm.org/show_bug.cgi?id=43871? 'Cause i'm super confused. Like, it's trivial to fix but i'm not sure what the correct behavior

[PATCH] D73629: [analyzer] vfork checker: allow execve after vfork

janvcelak updated this revision to Diff 241245. janvcelak added a comment. Sorry. I missed that step in contribution guidelines. I'm attaching updated version with larger context. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73629/new/ https://reviews.llvm.org/D73629 Files:

  1   2   >