[clang] [Support] Rename HashBuilderImpl to HashBuilder (NFC) (PR #68173)

2023-10-03 Thread Markus Böck via cfe-commits
https://github.com/zero9178 approved this pull request. https://github.com/llvm/llvm-project/pull/68173 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 7e856d1 - Reland "[clang-format] Annotate ctors/dtors as CtorDtorDeclName instead (#67955)"

2023-10-03 Thread via cfe-commits
Author: Owen Pan Date: 2023-10-03T22:26:48-07:00 New Revision: 7e856d18943f637b8c83f0bf8cbb506c5f7e94af URL: https://github.com/llvm/llvm-project/commit/7e856d18943f637b8c83f0bf8cbb506c5f7e94af DIFF: https://github.com/llvm/llvm-project/commit/7e856d18943f637b8c83f0bf8cbb506c5f7e94af.diff

[clang-tools-extra] [clang-tidy]: Add TagDecl into LastTagDeclRanges in UseUsingCheck only when it is a definition (PR #67639)

2023-10-03 Thread Piotr Zegar via cfe-commits
https://github.com/PiotrZSL approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/67639 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D155610: [Clang][Sema] Fix display of characters on static assertion failure

2023-10-03 Thread Takuya Shimizu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG2176c5e510e3: [Clang][Sema] Fix display of characters on static assertion failure (authored by hazohelet). Changed prior to commit: https://reviews.llvm.org/D155610?vs=557524=557581#toc Repository:

[clang] [clang] Preserve UDL nodes in RemoveNestedImmediateInvocation (PR #66641)

2023-10-03 Thread via cfe-commits
https://github.com/zyn0217 updated https://github.com/llvm/llvm-project/pull/66641 >From ef66dbe5f9a5b9071e994a5d8f2b6b48b6d5b446 Mon Sep 17 00:00:00 2001 From: Younan Zhang Date: Mon, 18 Sep 2023 19:55:45 +0800 Subject: [PATCH 1/2] [clang] Preserve UDL nodes in

[clang] 2176c5e - [Clang][Sema] Fix display of characters on static assertion failure

2023-10-03 Thread Takuya Shimizu via cfe-commits
Author: Takuya Shimizu Date: 2023-10-04T14:09:06+09:00 New Revision: 2176c5e510e3bfcbc75afb13e78d287141f239a7 URL: https://github.com/llvm/llvm-project/commit/2176c5e510e3bfcbc75afb13e78d287141f239a7 DIFF:

[clang] [clang][tidy] Ensure rewriter has the correct CWD (PR #67839)

2023-10-03 Thread Piotr Zegar via cfe-commits
https://github.com/PiotrZSL approved this pull request. Clang part looks fine. For a clang-tidy part, is there a way to test this part ? What changes because we use now a absolute build directory. I'm missing some tests for that part. https://github.com/llvm/llvm-project/pull/67839

[clang] [clang] Preserve UDL nodes in RemoveNestedImmediateInvocation (PR #66641)

2023-10-03 Thread via cfe-commits
zyn0217 wrote: No problem! And here it is. PTAL, thanks! @cor3ntin https://github.com/llvm/llvm-project/pull/66641 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Preserve UDL nodes in RemoveNestedImmediateInvocation (PR #66641)

2023-10-03 Thread via cfe-commits
https://github.com/zyn0217 updated https://github.com/llvm/llvm-project/pull/66641 >From ba9aa0ffe006ecdd8685c7e62be8c4c776391ec2 Mon Sep 17 00:00:00 2001 From: Younan Zhang Date: Mon, 18 Sep 2023 19:55:45 +0800 Subject: [PATCH 1/2] [clang] Preserve UDL nodes in

[clang-tools-extra] [clang-tidy][libc] Fix namespace check with macro (PR #68134)

2023-10-03 Thread Piotr Zegar via cfe-commits
https://github.com/PiotrZSL requested changes to this pull request. Order of changes, test, documentation is missing https://github.com/llvm/llvm-project/pull/68134 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang-tools-extra] [clang-tidy][libc] Fix namespace check with macro (PR #68134)

2023-10-03 Thread Piotr Zegar via cfe-commits
https://github.com/PiotrZSL edited https://github.com/llvm/llvm-project/pull/68134 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy][libc] Fix namespace check with macro (PR #68134)

2023-10-03 Thread Piotr Zegar via cfe-commits
@@ -45,18 +45,20 @@ void CalleeNamespaceCheck::check(const MatchFinder::MatchResult ) { if (FuncDecl->getBuiltinID() != 0) return; - // If the outermost namespace of the function is __llvm_libc, we're good. + // If the outermost namespace of the function starts with

[clang-tools-extra] [clang-tidy][libc] Fix namespace check with macro (PR #68134)

2023-10-03 Thread Piotr Zegar via cfe-commits
@@ -45,18 +45,20 @@ void CalleeNamespaceCheck::check(const MatchFinder::MatchResult ) { if (FuncDecl->getBuiltinID() != 0) return; - // If the outermost namespace of the function is __llvm_libc, we're good. + // If the outermost namespace of the function starts with

[clang-tools-extra] [clang-tidy][libc] Fix namespace check with macro (PR #68134)

2023-10-03 Thread Piotr Zegar via cfe-commits
@@ -321,6 +321,11 @@ Changes in existing checks ` check to identify calls to static member functions with out-of-class inline definitions. +- Improved :doc:`llvmlibc-callee-namespace PiotrZSL wrote: Put this in alphabetical order.

[clang-tools-extra] llvm-canon (PR #68176)

2023-10-03 Thread Justin Fargnoli via cfe-commits
https://github.com/justinfargnoli updated https://github.com/llvm/llvm-project/pull/68176 >From f792a030ac1761a96176332fea906cd2d1826c7b Mon Sep 17 00:00:00 2001 From: justinfargnoli Date: Sat, 12 Aug 2023 10:58:45 -0700 Subject: [PATCH 01/15] Add IRCanonicalizer.cpp ---

[clang] llvm-canon (PR #68176)

2023-10-03 Thread Justin Fargnoli via cfe-commits
https://github.com/justinfargnoli updated https://github.com/llvm/llvm-project/pull/68176 >From f792a030ac1761a96176332fea906cd2d1826c7b Mon Sep 17 00:00:00 2001 From: justinfargnoli Date: Sat, 12 Aug 2023 10:58:45 -0700 Subject: [PATCH 01/15] Add IRCanonicalizer.cpp ---

[clang-tools-extra] llvm-canon (PR #68176)

2023-10-03 Thread Michal Paszkowski via cfe-commits
@@ -0,0 +1,638 @@ +//===--- IRCanonicalizer.cpp - IR Canonicalizer ---===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier:

[clang] llvm-canon (PR #68176)

2023-10-03 Thread Michal Paszkowski via cfe-commits
@@ -0,0 +1,638 @@ +//===--- IRCanonicalizer.cpp - IR Canonicalizer ---===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier:

[clang-tools-extra] llvm-canon (PR #68176)

2023-10-03 Thread Michal Paszkowski via cfe-commits
michalpaszkowski wrote: Thank you @justinfargnoli and @AidanGoldfarb for taking over this work and reaching out! I will review your changes in the coming week. CC @ChrisCummins FYI https://github.com/llvm/llvm-project/pull/68176 ___ cfe-commits

[clang] llvm-canon (PR #68176)

2023-10-03 Thread Michal Paszkowski via cfe-commits
michalpaszkowski wrote: Thank you @justinfargnoli and @AidanGoldfarb for taking over this work and reaching out! I will review your changes in the coming week. CC @ChrisCummins FYI https://github.com/llvm/llvm-project/pull/68176 ___ cfe-commits

[clang-tools-extra] llvm-canon (PR #68176)

2023-10-03 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 8641cdf397d86f33ac45e4c691ca4f843c359370 90912d2b40121936c239f9275bebd52db5d3a116 --

[clang] llvm-canon (PR #68176)

2023-10-03 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 8641cdf397d86f33ac45e4c691ca4f843c359370 90912d2b40121936c239f9275bebd52db5d3a116 --

[clang-tools-extra] [clangd] Adapt Inlay Hint support for Deducing This (PR #68177)

2023-10-03 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 8641cdf397d86f33ac45e4c691ca4f843c359370 5e75d4d99cec39fdc3139f1e92036712aea90f57 --

[clang-tools-extra] [clang-tidy][libc] Fix namespace check with macro (PR #68134)

2023-10-03 Thread via cfe-commits
EugeneZelenko wrote: Shouldn't documentation be updated too? https://github.com/llvm/llvm-project/pull/68134 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clangd] Adapt Inlay Hint support for Deducing This (PR #68177)

2023-10-03 Thread via cfe-commits
https://github.com/zyn0217 updated https://github.com/llvm/llvm-project/pull/68177 >From 5e75d4d99cec39fdc3139f1e92036712aea90f57 Mon Sep 17 00:00:00 2001 From: Younan Zhang Date: Wed, 4 Oct 2023 11:59:31 +0800 Subject: [PATCH 1/2] [clangd] Adapt Inlay Hint support for Deducing This This is a

[clang-tools-extra] [clangd] Adapt Inlay Hint support for Deducing This (PR #68177)

2023-10-03 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clangd Changes This is a follow-up for D140828, making Clangd omit the explicit object parameter in a call to member function with Deducing This. Given that the parent patch is still in its infancy and might undergo several reverting-relanding

[clang-tools-extra] [clangd] Adapt Inlay Hint support for Deducing This (PR #68177)

2023-10-03 Thread via cfe-commits
https://github.com/zyn0217 created https://github.com/llvm/llvm-project/pull/68177 This is a follow-up for D140828, making Clangd omit the explicit object parameter in a call to member function with Deducing This. Given that the parent patch is still in its infancy and might undergo several

[clang] llvm-canon (PR #68176)

2023-10-03 Thread Justin Fargnoli via cfe-commits
https://github.com/justinfargnoli created https://github.com/llvm/llvm-project/pull/68176 Add the llvm-canon tool. Description from the [original PR](https://reviews.llvm.org/D66029#change-wZv3yOpDdxIu): > Added a new llvm-canon tool which aims to transform LLVM Modules into a > canonical

[clang] [Support] Rename HashBuilderImpl to HashBuilder (NFC) (PR #68173)

2023-10-03 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Changes Commit 9370271ec5debcb59e34629d5bd357c44213b2d3 made HashBuilder an alias for HashBuilderImpl: template class HasherT, support::endianness Endianness using HashBuilder = HashBuilderImplHasherT, Endianness; This patch renames

[clang] [Support] Rename HashBuilderImpl to HashBuilder (NFC) (PR #68173)

2023-10-03 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/68173 Commit 9370271ec5debcb59e34629d5bd357c44213b2d3 made HashBuilder an alias for HashBuilderImpl: template using HashBuilder = HashBuilderImpl; This patch renames HashBuilderImpl to HashBuilder while

[clang] Fixes and closes #53952. Setting the ASTHasCompilerErrors member variable correctly based on the PP diagnostics. (PR #68127)

2023-10-03 Thread Rajkumar Ananthu via cfe-commits
https://github.com/rajkumarananthu edited https://github.com/llvm/llvm-project/pull/68127 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Fixes and closes #53952. Setting the ASTHasCompilerErrors member variable correctly based on the PP diagnostics. (PR #68127)

2023-10-03 Thread Rajkumar Ananthu via cfe-commits
@@ -4628,6 +4628,12 @@ ASTFileSignature ASTWriter::WriteAST(Sema , StringRef OutputFile, WritingAST = true; ASTHasCompilerErrors = hasErrors; + bool trueHasErrors = SemaRef.PP.getDiagnostics().hasUncompilableErrorOccurred(); rajkumarananthu wrote: Hi

[clang] [Clang][Driver] Add new flags to control IR verification (PR #68172)

2023-10-03 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov edited https://github.com/llvm/llvm-project/pull/68172 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [CLang][Driver] Add new flags to control IR verification (PR #68172)

2023-10-03 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov created https://github.com/llvm/llvm-project/pull/68172 None >From 4aec8113bc23d361164f1e164c8e4e36a9eb0ad0 Mon Sep 17 00:00:00 2001 From: Matheus Izvekov Date: Wed, 4 Oct 2023 04:39:11 +0200 Subject: [PATCH] [CLang][Driver] Add new flags to control IR verification

[clang] [clang][Sema] Fix a bug when instantiating a lambda with requires clause (PR #65193)

2023-10-03 Thread via cfe-commits
https://github.com/0x59616e closed https://github.com/llvm/llvm-project/pull/65193 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 548d67a - [clang][Sema] Fix a bug when instantiating a lambda with requires clause (#65193)

2023-10-03 Thread via cfe-commits
Author: Sheng Date: 2023-10-04T10:19:35+08:00 New Revision: 548d67a0393c7bd200e335ada0a3d684750c2697 URL: https://github.com/llvm/llvm-project/commit/548d67a0393c7bd200e335ada0a3d684750c2697 DIFF: https://github.com/llvm/llvm-project/commit/548d67a0393c7bd200e335ada0a3d684750c2697.diff LOG:

[clang] [clang][Sema] Fix a bug when instantiating a lambda with requires clause (PR #65193)

2023-10-03 Thread via cfe-commits
0x59616e wrote: Thank you for your review. I appreciate your feedback ;) https://github.com/llvm/llvm-project/pull/65193 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Annotate ctors/dtors as CtorDtorDeclName instead (PR #67955)

2023-10-03 Thread Owen Pan via cfe-commits
owenca wrote: Reverted in d08fcc817eba. https://github.com/llvm/llvm-project/pull/67955 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] WebKit checkers: support ref and deref defined on different classes. (PR #68170)

2023-10-03 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Changes Patch by Ryosuke Niwa! --- Full diff: https://github.com/llvm/llvm-project/pull/68170.diff 5 Files Affected: - (modified) clang/lib/StaticAnalyzer/Checkers/WebKit/PtrTypesSemantics.cpp (+62-28) - (modified)

[clang] [analyzer] WebKit checkers: support ref and deref defined on different classes. (PR #68170)

2023-10-03 Thread Artem Dergachev via cfe-commits
https://github.com/haoNoQ created https://github.com/llvm/llvm-project/pull/68170 Patch by Ryosuke Niwa! >From fc5a447a0dd4203ee69a506cfc791255d555462a Mon Sep 17 00:00:00 2001 From: Ryosuke Niwa Date: Tue, 3 Oct 2023 18:13:21 -0700 Subject: [PATCH] [analyzer] WebKit checkers: support ref and

[clang] d08fcc8 - Revert "[clang-format] Annotate ctors/dtors as CtorDtorDeclName instead (#67955)"

2023-10-03 Thread Owen Pan via cfe-commits
Author: Owen Pan Date: 2023-10-03T18:19:23-07:00 New Revision: d08fcc817eba7f0186620688eee73f6d25fa90e8 URL: https://github.com/llvm/llvm-project/commit/d08fcc817eba7f0186620688eee73f6d25fa90e8 DIFF: https://github.com/llvm/llvm-project/commit/d08fcc817eba7f0186620688eee73f6d25fa90e8.diff

[clang] [clang-format] Annotate ctors/dtors as CtorDtorDeclName instead (PR #67955)

2023-10-03 Thread via cfe-commits
dyung wrote: I'm seeing the same failures on our linux/Windows build bots: https://lab.llvm.org/buildbot/#/builders/139/builds/50966 https://lab.llvm.org/buildbot/#/builders/216/builds/28315 https://github.com/llvm/llvm-project/pull/67955 ___

[clang] [Clang][CodeGen] Fix use of CXXThisValue with StrictVTablePointers (PR #68169)

2023-10-03 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-codegen Changes When emitting non-virtual base initializers for the constructor prologue, we would potentially use a re-laundered this pointer value from a previous block, which subsequently would not dominate this use. With this fix, we always

[clang] [Clang][CodeGen] Fix use of CXXThisValue with StrictVTablePointers (PR #68169)

2023-10-03 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov created https://github.com/llvm/llvm-project/pull/68169 When emitting non-virtual base initializers for the constructor prologue, we would potentially use a re-laundered this pointer value from a previous block, which subsequently would not dominate this use. With

[clang] [clang-format] Annotate ctors/dtors as CtorDtorDeclName instead (PR #67955)

2023-10-03 Thread Owen Pan via cfe-commits
https://github.com/owenca closed https://github.com/llvm/llvm-project/pull/67955 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 6a621ed - [clang-format] Annotate ctors/dtors as CtorDtorDeclName instead (#67955)

2023-10-03 Thread via cfe-commits
Author: Owen Pan Date: 2023-10-03T18:02:09-07:00 New Revision: 6a621ed8e4cb02bd55fe4a4a0254615576b70a55 URL: https://github.com/llvm/llvm-project/commit/6a621ed8e4cb02bd55fe4a4a0254615576b70a55 DIFF: https://github.com/llvm/llvm-project/commit/6a621ed8e4cb02bd55fe4a4a0254615576b70a55.diff

[clang-tools-extra] [libc++] Implement ranges::contains (PR #65148)

2023-10-03 Thread Konstantin Varlamov via cfe-commits
@@ -0,0 +1,61 @@ +//===--===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier:

[clang] [libc++] Implement ranges::contains (PR #65148)

2023-10-03 Thread Konstantin Varlamov via cfe-commits
@@ -0,0 +1,61 @@ +//===--===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier:

[libunwind] [libc++] Implement ranges::contains (PR #65148)

2023-10-03 Thread Konstantin Varlamov via cfe-commits
@@ -0,0 +1,61 @@ +//===--===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier:

[clang] Fixes and closes #53952. Setting the ASTHasCompilerErrors member variable correctly based on the PP diagnostics. (PR #68127)

2023-10-03 Thread Shafik Yaghmour via cfe-commits
@@ -4628,6 +4628,12 @@ ASTFileSignature ASTWriter::WriteAST(Sema , StringRef OutputFile, WritingAST = true; ASTHasCompilerErrors = hasErrors; + bool trueHasErrors = SemaRef.PP.getDiagnostics().hasUncompilableErrorOccurred(); shafik wrote: So this

[clang] Fixes and closes #53952. Setting the ASTHasCompilerErrors member variable correctly based on the PP diagnostics. (PR #68127)

2023-10-03 Thread Shafik Yaghmour via cfe-commits
https://github.com/shafik edited https://github.com/llvm/llvm-project/pull/68127 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Fixes and closes #53952. Setting the ASTHasCompilerErrors member variable correctly based on the PP diagnostics. (PR #68127)

2023-10-03 Thread Shafik Yaghmour via cfe-commits
https://github.com/shafik commented: The fix does not look correct but perhaps someone else will have more insight. https://github.com/llvm/llvm-project/pull/68127 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] 1dceba3 - [analyzer] Fix false negative when accessing a nonnull property from … (#67563)

2023-10-03 Thread via cfe-commits
Author: tripleCC Date: 2023-10-04T08:38:10+08:00 New Revision: 1dceba3a3684d12394731e09a6cf3efcebf07a3a URL: https://github.com/llvm/llvm-project/commit/1dceba3a3684d12394731e09a6cf3efcebf07a3a DIFF: https://github.com/llvm/llvm-project/commit/1dceba3a3684d12394731e09a6cf3efcebf07a3a.diff

[clang] [analyzer] Fix false negative when accessing a nonnull property from … (PR #67563)

2023-10-03 Thread via cfe-commits
https://github.com/tripleCC closed https://github.com/llvm/llvm-project/pull/67563 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D133361: [BPF] Attribute preserve_static_offset for structs

2023-10-03 Thread Eduard Zingerman via Phabricator via cfe-commits
eddyz87 added inline comments. Comment at: clang/lib/CodeGen/CGExpr.cpp:3701 + QualType PointeeType = E->getType()->getPointeeType(); + if (PointeeType.isNull()) +return false; erichkeane wrote: > We override `operator bool` to make this work. Sorry, just

[clang] [libc++] Implement ranges::contains (PR #65148)

2023-10-03 Thread via cfe-commits
https://github.com/ZijunZhaoCCK updated https://github.com/llvm/llvm-project/pull/65148 >From 02e9afd761228f401df4d9f8dfaaca44ffae0c6e Mon Sep 17 00:00:00 2001 From: zijunzhao Date: Thu, 31 Aug 2023 20:08:32 + Subject: [PATCH 01/12] [libc++] Implement ranges::contains Differential

[clang] 496d00b - [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR instead of External linkage (#67755)

2023-10-03 Thread via cfe-commits
Author: PiJoules Date: 2023-10-03T17:05:21-07:00 New Revision: 496d00b2677593066f5119e93cf0e9316a04277a URL: https://github.com/llvm/llvm-project/commit/496d00b2677593066f5119e93cf0e9316a04277a DIFF: https://github.com/llvm/llvm-project/commit/496d00b2677593066f5119e93cf0e9316a04277a.diff

[clang] [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR instead of External linkage (PR #67755)

2023-10-03 Thread via cfe-commits
https://github.com/PiJoules closed https://github.com/llvm/llvm-project/pull/67755 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR instead of External linkage (PR #67755)

2023-10-03 Thread via cfe-commits
https://github.com/PiJoules updated https://github.com/llvm/llvm-project/pull/67755 >From 2607a99be986215fa27c053755e96c81bb2033e7 Mon Sep 17 00:00:00 2001 From: Leonard Chan Date: Fri, 29 Sep 2023 00:25:44 + Subject: [PATCH] [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR

[clang] [clang][modules] Remove preloaded SLocEntries from PCM files (PR #66962)

2023-10-03 Thread Jan Svoboda via cfe-commits
https://github.com/jansvoboda11 edited https://github.com/llvm/llvm-project/pull/66962 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [libc++] Implement ranges::contains (PR #65148)

2023-10-03 Thread via cfe-commits
@@ -0,0 +1,61 @@ +//===--===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier:

[clang] [clang][modules] Remove preloaded SLocEntries from PCM files (PR #66962)

2023-10-03 Thread Jan Svoboda via cfe-commits
https://github.com/jansvoboda11 edited https://github.com/llvm/llvm-project/pull/66962 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D158561: [-Wunsafe-buffer-usage] Add AST info to the unclaimed DRE debug notes for analysis

2023-10-03 Thread Ziqing Luo via Phabricator via cfe-commits
ziqingluo-90 updated this revision to Diff 557579. ziqingluo-90 added a comment. address comments CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158561/new/ https://reviews.llvm.org/D158561 Files: clang/lib/Analysis/UnsafeBufferUsage.cpp

[clang] [IRPGO][ValueProfile] Instrument virtual table address that could be used to do virtual table address comparision for indirect-call-promotion. (PR #66825)

2023-10-03 Thread via cfe-commits
modiking wrote: > Yes there are tradeoffs to doing this purely with whole program class > hierarchy analysis vs with profiled type info, and in fact they can be > complementary. For example, the profile info can indicate what order to do > the vtable comparisons (i.e. descending order of

[clang] [IRPGO][ValueProfile] Instrument virtual table address that could be used to do virtual table address comparision for indirect-call-promotion. (PR #66825)

2023-10-03 Thread via cfe-commits
@@ -43,19 +45,15 @@ int main(int argc, const char *argv[]) { uint64_t bufsize = __llvm_profile_get_size_for_buffer_internal( __llvm_profile_begin_data(), __llvm_profile_end_data(), __llvm_profile_begin_counters(), __llvm_profile_end_counters(), -

[clang] [IRPGO][ValueProfile] Instrument virtual table address that could be used to do virtual table address comparision for indirect-call-promotion. (PR #66825)

2023-10-03 Thread via cfe-commits
@@ -19,20 +19,38 @@ namespace llvm { // Visitor class that finds all indirect call. struct PGOIndirectCallVisitor : public InstVisitor { std::vector IndirectCalls; + std::vector VTableAddrs; PGOIndirectCallVisitor() = default; void visitCallBase(CallBase ) { if

[clang] [mlir][sparse] Print new syntax (PR #68130)

2023-10-03 Thread Peiming Liu via cfe-commits
@@ -586,30 +586,56 @@ Attribute SparseTensorEncodingAttr::parse(AsmParser , Type type) { } void SparseTensorEncodingAttr::print(AsmPrinter ) const { - // Print the struct-like storage in dictionary fashion. - printer << "<{ lvlTypes = [ "; -

[clang] [mlir][sparse] Print new syntax (PR #68130)

2023-10-03 Thread Aart Bik via cfe-commits
@@ -586,30 +586,56 @@ Attribute SparseTensorEncodingAttr::parse(AsmParser , Type type) { } void SparseTensorEncodingAttr::print(AsmPrinter ) const { - // Print the struct-like storage in dictionary fashion. - printer << "<{ lvlTypes = [ "; -

[clang] [mlir][sparse] Print new syntax (PR #68130)

2023-10-03 Thread Aart Bik via cfe-commits
@@ -472,8 +472,11 @@ class SparseInsertGenerator llvm::raw_svector_ostream nameOstream(nameBuffer); nameOstream << kInsertFuncNamePrefix; const Level lvlRank = stt.getLvlRank(); -for (Level l = 0; l < lvlRank; l++) - nameOstream <<

[clang] [clang][modules] Remove preloaded SLocEntries from PCM files (PR #66962)

2023-10-03 Thread Jan Svoboda via cfe-commits
@@ -701,6 +701,10 @@ class SourceManager : public RefCountedBase { /// use (-ID - 2). SmallVector LoadedSLocEntryTable; + /// For each allocation in LoadedSLocEntryTable, we keep the new size. This + /// can be used to determine whether two FileIDs come from the same

[clang] [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR instead of External linkage (PR #67755)

2023-10-03 Thread Petr Hosek via cfe-commits
@@ -639,8 +639,16 @@ void CodeGenVTables::addRelativeComponent(ConstantArrayBuilder , // want the stub/proxy to be emitted for properly calculating the offset. // Examples where there would be no symbol emitted are available_externally // and private linkages. - auto

[clang] [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR instead of External linkage (PR #67755)

2023-10-03 Thread Petr Hosek via cfe-commits
@@ -639,8 +639,16 @@ void CodeGenVTables::addRelativeComponent(ConstantArrayBuilder , // want the stub/proxy to be emitted for properly calculating the offset. // Examples where there would be no symbol emitted are available_externally // and private linkages. - auto

[clang] Bugfix for chosing the correct deduction guide (PR #66487)

2023-10-03 Thread Shafik Yaghmour via cfe-commits
@@ -85,3 +85,13 @@ int main() { } + +namespace deduceTemplatedConstructor{ shafik wrote: I apologize for asking you to do additional work here but can you add the rest of the examples from [over.match.best.general

[clang] [PowerPC] Support local-dynamic TLS relocation on AIX (PR #66316)

2023-10-03 Thread Hubert Tong via cfe-commits
@@ -660,14 +671,16 @@ void PPCAsmPrinter::EmitTlsCall(const MachineInstr *MI, "GETtls[ld]ADDR[32] must read GPR3"); if (Subtarget->isAIXABI()) { -// On AIX, the variable offset should already be in R4 and the region handle -// should already be in R3. -

[clang] [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR instead of External linkage (PR #67755)

2023-10-03 Thread Petr Hosek via cfe-commits
https://github.com/petrhosek approved this pull request. https://github.com/llvm/llvm-project/pull/67755 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D159064: [Modules] Make clang modules for the C standard library headers

2023-10-03 Thread Med Ismail Bennani via Phabricator via cfe-commits
mib added a comment. Hey @iana, I think this broke the lldb bot https://green.lab.llvm.org/green/view/LLDB/job/as-lldb-cmake/6698/ Can you take a look ? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D159064/new/ https://reviews.llvm.org/D159064

[clang] [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR instead of External linkage (PR #67755)

2023-10-03 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 40c1f5b2fb88b17c6ed6911466002824d1880871 0e798f3fdd7dc8816a71e3e147ca14616ad4a123 --

[clang] Bugfix for chosing the correct deduction guide (PR #66487)

2023-10-03 Thread Shafik Yaghmour via cfe-commits
https://github.com/shafik approved this pull request. Thank you for the follow-up work. LGTM https://github.com/llvm/llvm-project/pull/66487 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [Inliner] Add argument/function attribute propagation before inlining. (PR #68164)

2023-10-03 Thread via cfe-commits
goldsteinn wrote: Note on compile time affect: https://llvm-compile-time-tracker.com/compare.php?from=2da4960f20f7e5d88a68ce25636a895284dc66d8=f8c9f5bce65756598da22e8aec5d91fb66b16d5c=instructions%3Au Minimal for normal O3, a bit more significant for LTO.

[clang] [Inliner] Add argument/function attribute propagation before inlining. (PR #68164)

2023-10-03 Thread via cfe-commits
https://github.com/goldsteinn updated https://github.com/llvm/llvm-project/pull/68164 >From 73290d5992e6a395fc953a9223173e9fd60a8775 Mon Sep 17 00:00:00 2001 From: Noah Goldstein Date: Thu, 7 Sep 2023 18:48:26 -0500 Subject: [PATCH 1/2] [Inliner] Propagate callee function memory access

[clang] [mlir][sparse] Print new syntax (PR #68130)

2023-10-03 Thread Yinying Li via cfe-commits
https://github.com/yinying-lisa-li updated https://github.com/llvm/llvm-project/pull/68130 >From 47b34bb327e1078678d3ba0c96ebce3fc89cf2ae Mon Sep 17 00:00:00 2001 From: Yinying Li Date: Tue, 3 Oct 2023 16:43:50 + Subject: [PATCH 1/4] [mlir][sparse] Print new syntax Printing changes from

[clang] Fix the Modules/compiler_builtins.m test (PR #68163)

2023-10-03 Thread Ian Anderson via cfe-commits
https://github.com/ian-twilightcoder closed https://github.com/llvm/llvm-project/pull/68163 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] b855ae9 - Fix the Modules/compiler_builtins.m test (#68163)

2023-10-03 Thread via cfe-commits
Author: Ian Anderson Date: 2023-10-03T15:58:38-07:00 New Revision: b855ae99036085267b4f7482f996c7d83d0d2983 URL: https://github.com/llvm/llvm-project/commit/b855ae99036085267b4f7482f996c7d83d0d2983 DIFF: https://github.com/llvm/llvm-project/commit/b855ae99036085267b4f7482f996c7d83d0d2983.diff

[clang] [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR instead of External linkage (PR #67755)

2023-10-03 Thread via cfe-commits
PiJoules wrote: > The change to `clang/test/CodeGenCXX/fixed-point-mangle.cpp` is unrelated. Woops. Yeah, removed. https://github.com/llvm/llvm-project/pull/67755 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR instead of External linkage (PR #67755)

2023-10-03 Thread via cfe-commits
https://github.com/PiJoules updated https://github.com/llvm/llvm-project/pull/67755 >From 0e798f3fdd7dc8816a71e3e147ca14616ad4a123 Mon Sep 17 00:00:00 2001 From: Leonard Chan Date: Fri, 29 Sep 2023 00:25:44 + Subject: [PATCH] [clang][RelativeVTables] Make the rtti_proxy LinkOnceODR

[clang] [mlir][sparse] Print new syntax (PR #68130)

2023-10-03 Thread Yinying Li via cfe-commits
@@ -533,7 +533,7 @@ func.func @sparse_compression(%tensor: tensor<8x8xf64, #CSR>, // CHECK: %[[A13:.*]]:4 = scf.for %[[A14:.*]] = %[[A11]] to %[[A7]] step %[[A12]] iter_args(%[[A15:.*]] = %[[A0]], %[[A16:.*]] = %[[A1]], %[[A17:.*]] = %[[A2]], %[[A18:.*]] = %[[A3]])

[clang] [mlir][sparse] Print new syntax (PR #68130)

2023-10-03 Thread Yinying Li via cfe-commits
https://github.com/yinying-lisa-li updated https://github.com/llvm/llvm-project/pull/68130 >From 47b34bb327e1078678d3ba0c96ebce3fc89cf2ae Mon Sep 17 00:00:00 2001 From: Yinying Li Date: Tue, 3 Oct 2023 16:43:50 + Subject: [PATCH 1/3] [mlir][sparse] Print new syntax Printing changes from

[clang-tools-extra] [OpenMPIRBuilder] Remove wrapper function in `createTask`, `createTeams` (PR #67723)

2023-10-03 Thread via cfe-commits
shraiysh wrote: Ping for review! https://github.com/llvm/llvm-project/pull/67723 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Ignore GCC 11 `[[malloc(x)]]` attribute (PR #68059)

2023-10-03 Thread Alois Klink via cfe-commits
@@ -4122,6 +4122,9 @@ def RestrictDocs : Documentation { The ``malloc`` attribute indicates that the function acts like a system memory allocation function, returning a pointer to allocated storage disjoint from the storage for any other object accessible to the caller. + +The

[clang] [OpenMPIRBuilder] Remove wrapper function in `createTask`, `createTeams` (PR #67723)

2023-10-03 Thread via cfe-commits
shraiysh wrote: Ping for review! https://github.com/llvm/llvm-project/pull/67723 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Inliner] Add argument/function attribute propagation before inlining. (PR #68164)

2023-10-03 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 1cfaa863bc36d25625114b432e2ddf35d2302452 f8c9f5bce65756598da22e8aec5d91fb66b16d5c --

[libunwind] [mlir][OpenMP] Added translation for `omp.teams` to LLVM IR (PR #68042)

2023-10-03 Thread via cfe-commits
https://github.com/shraiysh closed https://github.com/llvm/llvm-project/pull/68042 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libclc] [mlir][OpenMP] Added translation for `omp.teams` to LLVM IR (PR #68042)

2023-10-03 Thread via cfe-commits
https://github.com/shraiysh closed https://github.com/llvm/llvm-project/pull/68042 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy][libc] Fix namespace check with macro (PR #68134)

2023-10-03 Thread via cfe-commits
https://github.com/michaelrj-google updated https://github.com/llvm/llvm-project/pull/68134 >From baf9d8e19d2b064c05527757c6173f875b59b286 Mon Sep 17 00:00:00 2001 From: Michael Jones Date: Tue, 3 Oct 2023 10:39:02 -0700 Subject: [PATCH 1/2] [clang-tidy][libc] Fix namespace check with macro

[clang] Fix the Modules/compiler_builtins.m test (PR #68163)

2023-10-03 Thread Michael Spencer via cfe-commits
https://github.com/Bigcheese approved this pull request. https://github.com/llvm/llvm-project/pull/68163 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D148381: [Clang] Implement the 'counted_by' attribute

2023-10-03 Thread Bill Wendling via cfe-commits
Yes, I mean to do it as a direct follow-up.  -bw On Tue, Oct 3, 2023, 6:31 AM Aaron Ballman via Phabricator < revi...@reviews.llvm.org> wrote: > aaron.ballman added inline comments. > > > > Comment at: clang/lib/Sema/SemaDeclAttr.cpp:8420-8428 > +if (Result.getResultKind()

[clang] [Inliner] Add argument/function attribute propagation before inlining. (PR #68164)

2023-10-03 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Changes - [Inliner] Propagate callee function memory access attributes before inlining - [Inliner] Propagate callee argument memory access attributes before inlining --- Patch is 24.83 KiB, truncated to 20.00 KiB below, full version:

[clang] [Inliner] Add argument/function attribute propagation before inlining. (PR #68164)

2023-10-03 Thread via cfe-commits
https://github.com/goldsteinn edited https://github.com/llvm/llvm-project/pull/68164 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] goldsteinn/inliner attrs next next (PR #68164)

2023-10-03 Thread via cfe-commits
https://github.com/goldsteinn created https://github.com/llvm/llvm-project/pull/68164 - [Inliner] Propagate callee function memory access attributes before inlining - [Inliner] Propagate callee argument memory access attributes before inlining >From 73290d5992e6a395fc953a9223173e9fd60a8775

[clang-tools-extra] [clang-tidy][libc] Fix namespace check with macro (PR #68134)

2023-10-03 Thread Congcong Cai via cfe-commits
https://github.com/HerrCai0907 approved this pull request. LGTM. Please update the release notes. https://github.com/llvm/llvm-project/pull/68134 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

  1   2   3   4   5   >