[clang] [clang][bytecode] Implement fixed point casts (PR #110409)

2024-09-28 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/110409 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] c2a37e4 - [clang][bytecode] Implement fixed point casts (#110409)

2024-09-28 Thread via cfe-commits
Author: Timm Baeder Date: 2024-09-29T08:58:21+02:00 New Revision: c2a37e41b9f9f5e28339674dbdc656bc8a0bf708 URL: https://github.com/llvm/llvm-project/commit/c2a37e41b9f9f5e28339674dbdc656bc8a0bf708 DIFF: https://github.com/llvm/llvm-project/commit/c2a37e41b9f9f5e28339674dbdc656bc8a0bf708.diff L

[clang] [clang-format] Fix a bug in annotating angles containing FatArrow (PR #108671)

2024-09-28 Thread Owen Pan via cfe-commits
@@ -243,14 +244,16 @@ class AnnotatingParser { // operator that was misinterpreted because we are parsing template // parameters. // FIXME: This is getting out of hand, write a decent parser. - if (InExpr && !Line.startsWith(tok::kw_template) && + if

[clang] [llvm] [LoongArch] [CodeGen] Add options for Clang to generate LoongArch-specific frecipe & frsqrte instructions (PR #109917)

2024-09-28 Thread via cfe-commits
https://github.com/tangaac updated https://github.com/llvm/llvm-project/pull/109917 >From b720995dad35236f23681b3cc895f7aea9cc3147 Mon Sep 17 00:00:00 2001 From: tangaac Date: Wed, 25 Sep 2024 11:32:37 +0800 Subject: [PATCH 1/2] [LoongArch] Add options for Clang to generate LoongArch-specific

[clang] [NFC] Correct the misuse of the API in the Clang test-report script. (PR #108725)

2024-09-28 Thread via cfe-commits
c8ef wrote: Hi @tru @nikic, if the patch looks good to you, could you please land it for me? Thanks! https://github.com/llvm/llvm-project/pull/108725 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listin

[clang] [clang] Fix the local parameter of void type inside the `Requires` expression. (PR #109831)

2024-09-28 Thread via cfe-commits
c8ef wrote: ``` // C++23 [dcl.fct]p6: // An explicit-object-parameter-declaration is a parameter-declaration // with a this specifier. An explicit-object-parameter-declaration // shall appear only as the first parameter-declaration of a // parameter-declarati

[clang] [llvm] Adding splitdouble HLSL function (PR #109331)

2024-09-28 Thread via cfe-commits
@@ -3,12 +3,12 @@ // Supported targets // -// RUN: %clang -target dxil-unknown-shadermodel6.2-compute %s -S -o /dev/null 2>&1 | FileCheck --allow-empty --check-prefix=CHECK-VALID %s -// RUN: %clang -target spirv-unknown-vulkan-compute %s -S -o /dev/null 2>&1 | FileCheck --al

[clang] [clang] Fix the local parameter of void type inside the `Requires` expression. (PR #109831)

2024-09-28 Thread Younan Zhang via cfe-commits
zyn0217 wrote: > ``` > static_assert(requires(this int) { true; }); > ``` The grammar doesn't appear to disallow the explicit object parameter in a requires expression, see https://eel.is/c++draft/expr.prim.req#nt:requirement-parameter-list. But the usage is indeed suspicious to me. We probab

[clang] [clang][bytecode] Implement fixed point casts (PR #110409)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/110409.diff 5 Files Affected: - (modified) clang/lib/AST/ByteCode/Compiler.cpp (+8) - (modified) clang/lib/AST/ByteCode/FixedPoint.h (+5) - (modi

[clang] [clang][bytecode] Implement fixed point casts (PR #110409)

2024-09-28 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr created https://github.com/llvm/llvm-project/pull/110409 None >From 591335042fa3d0bb0fa732ece33a29e2b3565530 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timm=20B=C3=A4der?= Date: Sun, 29 Sep 2024 07:52:29 +0200 Subject: [PATCH] [clang][bytecode] Implement fixed point c

[clang] [clang-format] Handle template closer followed by empty paretheses (PR #110408)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-format Author: Owen Pan (owenca) Changes Fixes #109925. --- Full diff: https://github.com/llvm/llvm-project/pull/110408.diff 3 Files Affected: - (modified) clang/lib/Format/TokenAnnotator.cpp (+23-19) - (modified) clang/lib/Format/UnwrappedLin

[clang] [clang-format] Handle template closer followed by empty paretheses (PR #110408)

2024-09-28 Thread Owen Pan via cfe-commits
https://github.com/owenca created https://github.com/llvm/llvm-project/pull/110408 Fixes #109925. >From 27135c008868cc4f17b7ca5ff9c2af9c2fc02135 Mon Sep 17 00:00:00 2001 From: Owen Pan Date: Sat, 28 Sep 2024 22:42:56 -0700 Subject: [PATCH] [clang-format] Handle template closer followed by empt

[clang] [llvm] [clang][ci] Move libc++ testing into the main PR pipeline (PR #93318)

2024-09-28 Thread Mehdi Amini via cfe-commits
https://github.com/joker-eph edited https://github.com/llvm/llvm-project/pull/93318 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [clang][ci] Move libc++ testing into the main PR pipeline (PR #93318)

2024-09-28 Thread Mehdi Amini via cfe-commits
@@ -231,6 +245,10 @@ linux_projects_to_test=$(exclude-linux $(compute-projects-to-test ${modified_pro linux_check_targets=$(check-targets ${linux_projects_to_test} | sort | uniq) linux_projects=$(add-dependencies ${linux_projects_to_test} | sort | uniq) +linux_runtimes_to_tes

[clang] [clang] Fix the local parameter of void type inside the `Requires` expression. (PR #109831)

2024-09-28 Thread via cfe-commits
c8ef wrote: I have another simple question: If an explicit-object-parameter-declaration appears in the parameter list, should the requires expression evaluate to false? All three compilers I've tested seem to evaluate it to true, but I'm unsure if this is the correct behavior. ``` static_asse

[clang] [clang][bytecode] Implement fixed-point add (PR #110405)

2024-09-28 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/110405 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] defead4 - [clang][bytecode] Implement fixed-point add (#110405)

2024-09-28 Thread via cfe-commits
Author: Timm Baeder Date: 2024-09-29T07:13:17+02:00 New Revision: defead4d2dfc18fe423792005e99edc7b14edf2f URL: https://github.com/llvm/llvm-project/commit/defead4d2dfc18fe423792005e99edc7b14edf2f DIFF: https://github.com/llvm/llvm-project/commit/defead4d2dfc18fe423792005e99edc7b14edf2f.diff L

[clang] [llvm] [clang][ci] Move libc++ testing into the main PR pipeline (PR #93318)

2024-09-28 Thread Vlad Serebrennikov via cfe-commits
https://github.com/Endilll edited https://github.com/llvm/llvm-project/pull/93318 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [clang][ci] Move libc++ testing into the main PR pipeline (PR #93318)

2024-09-28 Thread Vlad Serebrennikov via cfe-commits
@@ -231,6 +245,10 @@ linux_projects_to_test=$(exclude-linux $(compute-projects-to-test ${modified_pro linux_check_targets=$(check-targets ${linux_projects_to_test} | sort | uniq) linux_projects=$(add-dependencies ${linux_projects_to_test} | sort | uniq) +linux_runtimes_to_tes

[clang] [clang] Finish implementation of P0522 (PR #96023)

2024-09-28 Thread Matheus Izvekov via cfe-commits
mizvekov wrote: ping https://github.com/llvm/llvm-project/pull/96023 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] Adding splitdouble HLSL function (PR #109331)

2024-09-28 Thread Chris B via cfe-commits
@@ -3,12 +3,12 @@ // Supported targets // -// RUN: %clang -target dxil-unknown-shadermodel6.2-compute %s -S -o /dev/null 2>&1 | FileCheck --allow-empty --check-prefix=CHECK-VALID %s -// RUN: %clang -target spirv-unknown-vulkan-compute %s -S -o /dev/null 2>&1 | FileCheck --al

[clang] [clang][bytecode] Implement fixed-point add (PR #110405)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/110405.diff 4 Files Affected: - (modified) clang/lib/AST/ByteCode/Compiler.cpp (+3) - (modified) clang/lib/AST/ByteCode/FixedPoint.h (+35-1) - (m

[clang] [clang][bytecode] Implement fixed-point add (PR #110405)

2024-09-28 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr created https://github.com/llvm/llvm-project/pull/110405 None >From 58de93b574d2b8e47d830b79a136521c10eca1c8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timm=20B=C3=A4der?= Date: Sun, 29 Sep 2024 06:12:01 +0200 Subject: [PATCH] [clang][bytecode] Implement fixed-point a

[clang-tools-extra] [clangd] Implement simple folding of preprocessor branches (PR #80592)

2024-09-28 Thread Tor Shepherd via cfe-commits
torshepherd wrote: Darn, this was just closed last week as I put up an issue https://github.com/llvm/llvm-project/issues/110390 😭 Clang-pseudo was just removed: https://github.com/llvm/llvm-project/pull/80081#issuecomment-2361060164 Is the plan to refactor now and re-put this up? https://g

[clang] [clang] Improve deduction of reference typed NTTP (PR #110393)

2024-09-28 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov edited https://github.com/llvm/llvm-project/pull/110393 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [ItaniumMangle] Add substitutions for record types when mangling vtables (PR #109970)

2024-09-28 Thread via cfe-commits
https://github.com/tcwzxx closed https://github.com/llvm/llvm-project/pull/109970 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 7883b02 - [ItaniumMangle] Add substitutions for record types when mangling vtables (#109970)

2024-09-28 Thread via cfe-commits
Author: tcwzxx Date: 2024-09-29T11:43:28+08:00 New Revision: 7883b028b42df7b763cae20d8ff56233bee4beb6 URL: https://github.com/llvm/llvm-project/commit/7883b028b42df7b763cae20d8ff56233bee4beb6 DIFF: https://github.com/llvm/llvm-project/commit/7883b028b42df7b763cae20d8ff56233bee4beb6.diff LOG: [

[clang] [ItaniumMangle] Add substitutions for record types when mangling vtables (PR #109970)

2024-09-28 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov edited https://github.com/llvm/llvm-project/pull/109970 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Improve deduction of reference typed NTTP (PR #110393)

2024-09-28 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov updated https://github.com/llvm/llvm-project/pull/110393 >From a06cdbc7a2d010a3f3081b7eb53326aeb7a702b1 Mon Sep 17 00:00:00 2001 From: Matheus Izvekov Date: Sat, 28 Sep 2024 21:40:54 -0300 Subject: [PATCH] [clang] Improve deduction of reference typed NTTP This impro

[clang] [clang] Redeclare function templates instances per primary template (PR #110387)

2024-09-28 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov updated https://github.com/llvm/llvm-project/pull/110387 >From ee9ef305403cbcdf004e9777feab44689bf579a4 Mon Sep 17 00:00:00 2001 From: Matheus Izvekov Date: Sat, 28 Sep 2024 14:28:58 -0300 Subject: [PATCH] [clang] Redeclare function templates instances per primary t

[clang] bf9ab0b - [C++20] [Modules] Emit implicit Deduction Guide for implicit class specialization

2024-09-28 Thread Chuanqi Xu via cfe-commits
Author: Chuanqi Xu Date: 2024-09-29T10:38:05+08:00 New Revision: bf9ab0b7c3ba1d4a89a3dd9b00909ad3cfab3d48 URL: https://github.com/llvm/llvm-project/commit/bf9ab0b7c3ba1d4a89a3dd9b00909ad3cfab3d48 DIFF: https://github.com/llvm/llvm-project/commit/bf9ab0b7c3ba1d4a89a3dd9b00909ad3cfab3d48.diff LO

[clang-tools-extra] [clangd] [C++20] [Modules] Support code complete for C++20 modules (PR #110083)

2024-09-28 Thread Chuanqi Xu via cfe-commits
@@ -2108,7 +2116,7 @@ class CodeCompleteFlow { } // namespace -clang::CodeCompleteOptions CodeCompleteOptions::getClangCompleteOpts() const { +clang::CodeCompleteOptions CodeCompleteOptions::getClangCompleteOpts(bool ForceLoadExternal) const { ChuanqiXu9 wro

[clang-tools-extra] [clangd] [C++20] [Modules] Support code complete for C++20 modules (PR #110083)

2024-09-28 Thread Chuanqi Xu via cfe-commits
https://github.com/ChuanqiXu9 updated https://github.com/llvm/llvm-project/pull/110083 >From e35e600159c99736de7d2bc735c738002f592988 Mon Sep 17 00:00:00 2001 From: Chuanqi Xu Date: Thu, 26 Sep 2024 13:43:51 +0800 Subject: [PATCH 1/3] [clangd] [C++20] [Modules] Support code complete for C++20

[clang] [llvm] [LoongArch] [CodeGen] Add options for Clang to generate LoongArch-specific frecipe & frsqrte instructions (PR #109917)

2024-09-28 Thread via cfe-commits
@@ -251,6 +251,20 @@ void loongarch::getLoongArchTargetFeatures(const Driver &D, } else /*-mno-lasx*/ Features.push_back("-lasx"); } + + // Select frecipe feature determined by -m[no-]frecipe. + if (const Arg *A = + Args.getLastArg(options::OPT_mfrecipe,

[clang] [libcxx] Revert "[libc++][modules] Rewrite the modulemap to have fewer top-level modules (#107638)" (PR #110384)

2024-09-28 Thread Anton Korobeynikov via cfe-commits
asl wrote: @llvm-beanz The changes to clang/include/clang/AST/Expr.h seems to be unrelated, no? https://github.com/llvm/llvm-project/pull/110384 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cf

[clang] [llvm] workflows/release-binaries: Use static ZSTD on macOS (PR #109909)

2024-09-28 Thread Keith Smiley via cfe-commits
keith wrote: I pushed a dummy change to that just to verify so we don't have to go back and forth on it again. I put up the PR for the path changes here https://github.com/llvm/llvm-project/pull/110394, I was surprised to see this file is in the clang/ specific subdir, but also didn't see any

[clang] [llvm] [cuda][HIP] `__constant__` should imply constant (PR #110182)

2024-09-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-x86_64-linux-android` running on `sanitizer-buildbot-android` while building `clang,llvm` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/186/builds/2761 Here is the r

[clang] [llvm] workflows/release-binaries: Use static ZSTD on macOS (PR #109909)

2024-09-28 Thread Keith Smiley via cfe-commits
https://github.com/keith updated https://github.com/llvm/llvm-project/pull/109909 >From 4676d022d971ef1623b83a2731ce94572905d6ad Mon Sep 17 00:00:00 2001 From: Keith Smiley Date: Wed, 25 Sep 2024 05:38:04 + Subject: [PATCH 1/3] workflows/release-binaries: Use static ZSTD on macOS On macOS

[clang] [clang] Improve deduction of reference typed NTTP (PR #110393)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Matheus Izvekov (mizvekov) Changes This improves the existing workaround for a core issue introduced in CWG1770. When performing template argument deduction for an NNTP which the parameter side is a reference, instead of dropping the refe

[clang] [clang] Improve deduction of reference typed NTTP (PR #110393)

2024-09-28 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov created https://github.com/llvm/llvm-project/pull/110393 This improves the existing workaround for a core issue introduced in CWG1770. When performing template argument deduction for an NNTP which the parameter side is a reference, instead of dropping the references

[clang] [llvm] [cuda][HIP] `__constant__` should imply constant (PR #110182)

2024-09-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `openmp-s390x-linux` running on `systemz-1` while building `clang,llvm` at step 6 "test-openmp". Full details are available at: https://lab.llvm.org/buildbot/#/builders/88/builds/2915 Here is the relevant piece of the build l

[clang] [llvm] Adding splitdouble HLSL function (PR #109331)

2024-09-28 Thread via cfe-commits
@@ -3,12 +3,12 @@ // Supported targets // -// RUN: %clang -target dxil-unknown-shadermodel6.2-compute %s -S -o /dev/null 2>&1 | FileCheck --allow-empty --check-prefix=CHECK-VALID %s -// RUN: %clang -target spirv-unknown-vulkan-compute %s -S -o /dev/null 2>&1 | FileCheck --al

[clang] [llvm] [cuda][HIP] `__constant__` should imply constant (PR #110182)

2024-09-28 Thread Alex Voicu via cfe-commits
https://github.com/AlexVlx closed https://github.com/llvm/llvm-project/pull/110182 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] e203a67 - [cuda][HIP] `__constant__` should imply constant (#110182)

2024-09-28 Thread via cfe-commits
Author: Alex Voicu Date: 2024-09-29T01:22:52+01:00 New Revision: e203a67f4cef5844877f6a5720e659ea09729e9a URL: https://github.com/llvm/llvm-project/commit/e203a67f4cef5844877f6a5720e659ea09729e9a DIFF: https://github.com/llvm/llvm-project/commit/e203a67f4cef5844877f6a5720e659ea09729e9a.diff LO

[clang] [llvm] workflows/release-binaries: Use static ZSTD on macOS (PR #109909)

2024-09-28 Thread Tom Stellard via cfe-commits
https://github.com/tstellar approved this pull request. This looks good to me. It looks like we need to add the Release.cmake file to the list of files that trigger the release-binaries-all test, but that can be done in another PR. https://github.com/llvm/llvm-project/pull/109909

[clang] [libcxx] Revert "[libc++][modules] Rewrite the modulemap to have fewer top-level modules (#107638)" (PR #110384)

2024-09-28 Thread Aiden Grossman via cfe-commits
https://github.com/boomanaiden154 approved this pull request. LGTM, assuming precommit CI passes. https://github.com/llvm/llvm-project/pull/110384 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/c

[clang] [clang] Redeclare function templates instances per primary template (PR #110387)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Matheus Izvekov (mizvekov) Changes This fixes handling of friend function templates instances when their template context changes, such as when a new friend declaration is introduced after an instance was already created from a previous d

[clang] [clang] Redeclare function templates instances per primary template (PR #110387)

2024-09-28 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov created https://github.com/llvm/llvm-project/pull/110387 This fixes handling of friend function templates instances when their template context changes, such as when a new friend declaration is introduced after an instance was already created from a previous declara

[clang] [clang] Fix the local parameter of void type inside the `Requires` expression. (PR #109831)

2024-09-28 Thread Matheus Izvekov via cfe-commits
mizvekov wrote: Same then, go ahead, but please make sure the error handling strategies don't diverge unnecessarily. https://github.com/llvm/llvm-project/pull/109831 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bi

[clang-tools-extra] [clang-tidy][readability-container-contains] Fix matching of non-binaryOperator cases (PR #110386)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tidy @llvm/pr-subscribers-clang-tools-extra Author: Nicolas van Kempen (nicovank) Changes Fix #79437. It works with non-mock `std::map`: ``` # All cases detailed in #79437. > cat tmp.cpp #include bool a(std::map &m, int key) { ret

[clang-tools-extra] [clang-tidy][readability-container-contains] Fix matching of non-binaryOperator cases (PR #110386)

2024-09-28 Thread Nicolas van Kempen via cfe-commits
https://github.com/nicovank created https://github.com/llvm/llvm-project/pull/110386 Fix #79437. It works with non-mock `std::map`: ``` # All cases detailed in #79437. > cat tmp.cpp #include bool a(std::map &m, int key) { return m.find(key) != m.end(); } bool b(std::map &m, int key) { return

[clang] [llvm] [clang][ci] Move libc++ testing into the main PR pipeline (PR #93318)

2024-09-28 Thread Mehdi Amini via cfe-commits
@@ -231,6 +245,10 @@ linux_projects_to_test=$(exclude-linux $(compute-projects-to-test ${modified_pro linux_check_targets=$(check-targets ${linux_projects_to_test} | sort | uniq) linux_projects=$(add-dependencies ${linux_projects_to_test} | sort | uniq) +linux_runtimes_to_tes

[clang-tools-extra] [clangd] Add inlay hints for default function arguments (PR #95712)

2024-09-28 Thread Tor Shepherd via cfe-commits
https://github.com/torshepherd edited https://github.com/llvm/llvm-project/pull/95712 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clangd] Add inlay hints for default function arguments (PR #95712)

2024-09-28 Thread Tor Shepherd via cfe-commits
https://github.com/torshepherd edited https://github.com/llvm/llvm-project/pull/95712 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] support string::contains (PR #110351)

2024-09-28 Thread Nicolas van Kempen via cfe-commits
@@ -94,12 +102,14 @@ void ContainerContainsCheck::registerMatchers(MatchFinder *Finder) { binaryOperator(hasLHS(Literal1), hasOperatorName(">"), hasRHS(CountCall)) .bind("negativeComparison")); - // Find membership tests based on `find() == end()`. + // Find

[clang-tools-extra] [clang-tidy] support string::contains (PR #110351)

2024-09-28 Thread Nicolas van Kempen via cfe-commits
@@ -453,3 +490,15 @@ void testOperandPermutations(std::map& Map) { // CHECK-MESSAGES: :[[@LINE-1]]:{{[0-9]+}}: warning: use 'contains' to check for membership [readability-container-contains] // CHECK-FIXES: if (!Map.contains(0)) {}; } + +void testStringNops(std::string St

[clang-tools-extra] [clang-tidy] support string::contains (PR #110351)

2024-09-28 Thread Nicolas van Kempen via cfe-commits
@@ -29,6 +29,43 @@ struct multimap { bool contains(const Key &K) const; }; +using size_t = decltype(sizeof(int)); + +// Lightweight standin for std::string_view. +template +class basic_string_view { +public: + basic_string_view(); + basic_string_view(const basic_string_vi

[clang-tools-extra] [clang-tidy] support string::contains (PR #110351)

2024-09-28 Thread Nicolas van Kempen via cfe-commits
@@ -51,6 +53,12 @@ void ContainerContainsCheck::registerMatchers(MatchFinder *Finder) { const auto Literal0 = integerLiteral(equals(0)); const auto Literal1 = integerLiteral(equals(1)); + const auto StringLikeClass = cxxRecordDecl( + hasAnyName("::std::basic_string"

[clang-tools-extra] [clang-tidy] support string::contains (PR #110351)

2024-09-28 Thread Nicolas van Kempen via cfe-commits
https://github.com/nicovank commented: IMO no need to update release notes: > Improved `readability-container-contains` check to let it work on any class > that has a contains method. But I think an example should be added to [the check documentation](https://github.com/llvm/llvm-project/blob

[clang-tools-extra] [clang-tidy] support string::contains (PR #110351)

2024-09-28 Thread Nicolas van Kempen via cfe-commits
@@ -32,7 +33,8 @@ void ContainerContainsCheck::registerMatchers(MatchFinder *Finder) { const auto FindCall = cxxMemberCallExpr( - argumentCountIs(1), + anyOf(argumentCountIs(1), +allOf(argumentCountIs(2), hasArgument(1, cxxDefaultArg

[clang-tools-extra] [clang-tidy] support string::contains (PR #110351)

2024-09-28 Thread Nicolas van Kempen via cfe-commits
https://github.com/nicovank edited https://github.com/llvm/llvm-project/pull/110351 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [Utils] Add new --update-tests flag to llvm-lit (PR #108425)

2024-09-28 Thread Henrik G. Olsson via cfe-commits
hnrklssn wrote: > Thank you. This pretty much LGTM, but it just occurred to me that there's a > docs/CommandGuide/lit.rst which should be updated to document the new option. Done > As update-verify-tests was reverted, can you rebase this to just the > update_test_checks support? On the LLVM s

[clang] [llvm] [Utils] Add new --update-tests flag to llvm-lit (PR #108425)

2024-09-28 Thread Henrik G. Olsson via cfe-commits
https://github.com/hnrklssn updated https://github.com/llvm/llvm-project/pull/108425 >From 6cdb6bec945725d69d1073deeb53b7485c7e40cd Mon Sep 17 00:00:00 2001 From: "Henrik G. Olsson" Date: Wed, 28 Aug 2024 23:30:49 -0700 Subject: [PATCH 1/3] [Utils] Add --update-tests to lit This adds a flag to

[clang] [llvm] [Utils] Add new --update-tests flag to llvm-lit (PR #108425)

2024-09-28 Thread Henrik G. Olsson via cfe-commits
https://github.com/hnrklssn updated https://github.com/llvm/llvm-project/pull/108425 >From 6cdb6bec945725d69d1073deeb53b7485c7e40cd Mon Sep 17 00:00:00 2001 From: "Henrik G. Olsson" Date: Wed, 28 Aug 2024 23:30:49 -0700 Subject: [PATCH 1/2] [Utils] Add --update-tests to lit This adds a flag to

[clang] [clang-format] Add an option to control indentation of `export { ... }` (PR #110381)

2024-09-28 Thread via cfe-commits
https://github.com/Sirraide updated https://github.com/llvm/llvm-project/pull/110381 >From 570aff7459311a43cd9c9139de05dc2ab4cf762c Mon Sep 17 00:00:00 2001 From: Sirraide Date: Sat, 28 Sep 2024 20:36:38 +0200 Subject: [PATCH 1/3] [clang-format] Add an option to control indentation of `export

[clang] [llvm] Adding splitdouble HLSL function (PR #109331)

2024-09-28 Thread via cfe-commits
https://github.com/joaosaffran updated https://github.com/llvm/llvm-project/pull/109331 >From ef969c536d700a8585f0892952fae49cdd9c42d1 Mon Sep 17 00:00:00 2001 From: Joao Saffran Date: Thu, 19 Sep 2024 00:13:51 + Subject: [PATCH 01/10] Codegen builtin --- clang/include/clang/Basic/Builtin

[clang] [clang-format] Add an option to control indentation of `export { ... }` (PR #110381)

2024-09-28 Thread via cfe-commits
https://github.com/Sirraide created https://github.com/llvm/llvm-project/pull/110381 `export { ... }` blocks can get a bit long, so I thought it would make sense to have an option that makes it so their contents are not indented (basically the same argument as for namespaces). This is my first

[clang] [clang-format] Add an option to control indentation of `export { ... }` (PR #110381)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-format Author: None (Sirraide) Changes `export { ... }` blocks can get a bit long, so I thought it would make sense to have an option that makes it so their contents are not indented (basically the same argument as for namespaces). This is my fir

[clang] [llvm] Adding splitdouble HLSL function (PR #109331)

2024-09-28 Thread via cfe-commits
@@ -438,6 +438,55 @@ template constexpr uint asuint(T F) { return __detail::bit_cast(F); } +//===--===// +// asuint splitdouble builtins +//===---

[clang] [llvm] Adding splitdouble HLSL function (PR #109331)

2024-09-28 Thread via cfe-commits
https://github.com/joaosaffran updated https://github.com/llvm/llvm-project/pull/109331 >From ef969c536d700a8585f0892952fae49cdd9c42d1 Mon Sep 17 00:00:00 2001 From: Joao Saffran Date: Thu, 19 Sep 2024 00:13:51 + Subject: [PATCH 01/10] Codegen builtin --- clang/include/clang/Basic/Builtin

[clang] [llvm] Adding splitdouble HLSL function (PR #109331)

2024-09-28 Thread via cfe-commits
https://github.com/joaosaffran updated https://github.com/llvm/llvm-project/pull/109331 >From ef969c536d700a8585f0892952fae49cdd9c42d1 Mon Sep 17 00:00:00 2001 From: Joao Saffran Date: Thu, 19 Sep 2024 00:13:51 + Subject: [PATCH 01/10] Codegen builtin --- clang/include/clang/Basic/Builtin

[clang] [llvm] [cuda][HIP] `__constant__` should imply constant (PR #110182)

2024-09-28 Thread Alex Voicu via cfe-commits
https://github.com/AlexVlx updated https://github.com/llvm/llvm-project/pull/110182 >From af1adfafaa09bc7992cf9aaf34a6121cf2d56d5b Mon Sep 17 00:00:00 2001 From: Alex Voicu Date: Thu, 26 Sep 2024 04:16:52 +0100 Subject: [PATCH 1/3] Mark globals as `constant` if they have been annotated with `_

[clang] [RISCV][FMV] Support target_version (PR #99040)

2024-09-28 Thread Craig Topper via cfe-commits
https://github.com/topperc approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/99040 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] Adding splitdouble HLSL function (PR #109331)

2024-09-28 Thread Chris B via cfe-commits
@@ -438,6 +438,55 @@ template constexpr uint asuint(T F) { return __detail::bit_cast(F); } +//===--===// +// asuint splitdouble builtins +//===---

[clang] [llvm] Adding splitdouble HLSL function (PR #109331)

2024-09-28 Thread Chris B via cfe-commits
@@ -3,12 +3,12 @@ // Supported targets // -// RUN: %clang -target dxil-unknown-shadermodel6.2-compute %s -S -o /dev/null 2>&1 | FileCheck --allow-empty --check-prefix=CHECK-VALID %s -// RUN: %clang -target spirv-unknown-vulkan-compute %s -S -o /dev/null 2>&1 | FileCheck --al

[clang] [llvm] Adding splitdouble HLSL function (PR #109331)

2024-09-28 Thread Chris B via cfe-commits
@@ -422,6 +422,55 @@ template constexpr uint asuint(T F) { return __detail::bit_cast(F); } +//===--===// +// asuint splitdouble builtins +//===---

[clang] [Clang] Update Interpreter tests to use clang_target_link_libraries (PR #110154)

2024-09-28 Thread Thomas Fransham via cfe-commits
https://github.com/fsfod updated https://github.com/llvm/llvm-project/pull/110154 >From e58b244f90c0d9863a41721b7c981d5d853b3d38 Mon Sep 17 00:00:00 2001 From: Thomas Fransham Date: Thu, 26 Sep 2024 18:59:33 +0100 Subject: [PATCH] [Clang] Update Interpreter tests to use clang_target_link_libra

[clang] [clang][bytecode] Implement fixed-point-to-float casts (PR #110369)

2024-09-28 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/110369 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 3a5b9da - [clang][bytecode] Implement fixed-point-to-float casts (#110369)

2024-09-28 Thread via cfe-commits
Author: Timm Baeder Date: 2024-09-28T19:11:49+02:00 New Revision: 3a5b9da11598e714890f4e9f37f894fc2c7287c2 URL: https://github.com/llvm/llvm-project/commit/3a5b9da11598e714890f4e9f37f894fc2c7287c2 DIFF: https://github.com/llvm/llvm-project/commit/3a5b9da11598e714890f4e9f37f894fc2c7287c2.diff L

[clang] [clang][x86] Add constexpr support for BMI/TBM BEXTR intrinsics (PR #109577)

2024-09-28 Thread Simon Pilgrim via cfe-commits
https://github.com/RKSimon closed https://github.com/llvm/llvm-project/pull/109577 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 1c2ed36 - [clang][x86] Add constexpr support for BMI/TBM BEXTR intrinsics (#109577)

2024-09-28 Thread via cfe-commits
Author: Simon Pilgrim Date: 2024-09-28T17:52:54+01:00 New Revision: 1c2ed36741c6bc3921d2566123019079c4249c1a URL: https://github.com/llvm/llvm-project/commit/1c2ed36741c6bc3921d2566123019079c4249c1a DIFF: https://github.com/llvm/llvm-project/commit/1c2ed36741c6bc3921d2566123019079c4249c1a.diff

[clang] [llvm] workflows/release-binaries: Use static ZSTD on macOS (PR #109909)

2024-09-28 Thread Keith Smiley via cfe-commits
@@ -126,7 +126,7 @@ jobs: # add extra CMake args to disable them. # See https://github.com/llvm/llvm-project/issues/99767 if [ "${{ runner.os }}" = "macOS" ]; then - target_cmake_flags="$target_cmake_flags -DBOOTSTRAP_COMPILER_RT_ENABLE_IOS=OFF

[clang] [llvm] workflows/release-binaries: Use static ZSTD on macOS (PR #109909)

2024-09-28 Thread Keith Smiley via cfe-commits
https://github.com/keith updated https://github.com/llvm/llvm-project/pull/109909 >From 4676d022d971ef1623b83a2731ce94572905d6ad Mon Sep 17 00:00:00 2001 From: Keith Smiley Date: Wed, 25 Sep 2024 05:38:04 + Subject: [PATCH 1/2] workflows/release-binaries: Use static ZSTD on macOS On macOS

[clang] [Clang] prevent recovery call expression from proceeding with explicit attributes and undeclared templates (PR #107786)

2024-09-28 Thread Oleksandr T. via cfe-commits
https://github.com/a-tarasyuk updated https://github.com/llvm/llvm-project/pull/107786 >From e9948a1004cc2b486a0422d83e88392754e9f7e9 Mon Sep 17 00:00:00 2001 From: Oleksandr T Date: Mon, 23 Sep 2024 17:17:30 +0300 Subject: [PATCH 1/2] [Clang] prevent recovery call expression from proceeding w

[clang] [clang][bytecode] Implement fixed-point-to-float casts (PR #110369)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/110369.diff 5 Files Affected: - (modified) clang/lib/AST/ByteCode/Compiler.cpp (+6) - (modified) clang/lib/AST/ByteCode/FixedPoint.h (+4) - (modi

[clang] [clang][bytecode] Implement fixed-point-to-float casts (PR #110369)

2024-09-28 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr created https://github.com/llvm/llvm-project/pull/110369 None >From af2fe551a5324cd1274e1da8d6a14303646ae5b6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timm=20B=C3=A4der?= Date: Sat, 28 Sep 2024 16:02:56 +0200 Subject: [PATCH] [clang][bytecode] Implement fixed-point-t

[clang] [clang][bytecode] Implement floating-to-fixed-point casts (PR #110361)

2024-09-28 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/110361 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 6cbd8a3 - [clang][bytecode] Implement floating-to-fixed-point casts (#110361)

2024-09-28 Thread via cfe-commits
Author: Timm Baeder Date: 2024-09-28T17:24:28+02:00 New Revision: 6cbd8a309485329a4fbfe7abf7b85e0b8f154561 URL: https://github.com/llvm/llvm-project/commit/6cbd8a309485329a4fbfe7abf7b85e0b8f154561 DIFF: https://github.com/llvm/llvm-project/commit/6cbd8a309485329a4fbfe7abf7b85e0b8f154561.diff L

[clang] [clang] Fix the local parameter of void type inside the `Requires` expression. (PR #109831)

2024-09-28 Thread Younan Zhang via cfe-commits
zyn0217 wrote: > Therefore, I recommend adding the void parameter check directly into > SemaExprCXX for an immediate fix. If there's a future need to consolidate the > checks, we can consider refactoring later. That works for me if it turns out to take much more effort than our anticipation,

[clang] [clang-tools-extra] [clang-tidy] add `ctime` and `localtime` to `clang-tidy` (PR #110366)

2024-09-28 Thread Зишан Мирза via cfe-commits
https://github.com/zimirza updated https://github.com/llvm/llvm-project/pull/110366 From 4fb69942effb3cf34d07f33a14a95757b6ca5ee0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=97=D0=B8=D1=88=D0=B0=D0=BD=20=D0=9C=D0=B8=D1=80=D0=B7?= =?UTF-8?q?=D0=B0?= Date: Sat, 28 Sep 2024 17:05:42 +0200 Subjec

[clang] [Clang][Concepts] Normalize SizeOfPackExpr's pack declaration (PR #110238)

2024-09-28 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 updated https://github.com/llvm/llvm-project/pull/110238 >From 23a765ac6d8e455121346405332d2066dcc0861e Mon Sep 17 00:00:00 2001 From: Younan Zhang Date: Fri, 27 Sep 2024 18:23:47 +0800 Subject: [PATCH 1/7] [Clang] GH93099 --- clang/include/clang/Sema/Sema.h

[clang] [clang-tidy] add `ctime` and `localtime` to `clang-tidy` (PR #110366)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-static-analyzer-1 Author: Зишан Мирза (zimirza) Changes Closes #107445 --- Full diff: https://github.com/llvm/llvm-project/pull/110366.diff 5 Files Affected: - (modified) clang/docs/tools/clang-formatted-files.txt (+8) - (modified) clang/lib/S

[clang] [clang-tidy] add `ctime` and `localtime` to `clang-tidy` (PR #110366)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Зишан Мирза (zimirza) Changes Closes #107445 --- Full diff: https://github.com/llvm/llvm-project/pull/110366.diff 5 Files Affected: - (modified) clang/docs/tools/clang-formatted-files.txt (+8) - (modified) clang/lib/StaticAnalyzer/Chec

[clang] [clang-tidy] add `ctime` and `localtime` to `clang-tidy` (PR #110366)

2024-09-28 Thread Зишан Мирза via cfe-commits
https://github.com/zimirza created https://github.com/llvm/llvm-project/pull/110366 Closes #107445 From 4fb69942effb3cf34d07f33a14a95757b6ca5ee0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=97=D0=B8=D1=88=D0=B0=D0=BD=20=D0=9C=D0=B8=D1=80=D0=B7?= =?UTF-8?q?=D0=B0?= Date: Sat, 28 Sep 2024 17:05

[clang] [Clang][Concepts] Normalize SizeOfPackExpr's pack declaration (PR #110238)

2024-09-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Younan Zhang (zyn0217) Changes SizeOfPackExpr has a pointer to the referenced pack declaration, which is left as-is during the transformation process. The situation could be subtle when a friend class template declaration comes into play

[clang] [Clang][Concepts] Normalize SizeOfPackExpr's pack declaration (PR #110238)

2024-09-28 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 ready_for_review https://github.com/llvm/llvm-project/pull/110238 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Concepts] Normalize SizeOfPackExpr's pack declaration (PR #110238)

2024-09-28 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 edited https://github.com/llvm/llvm-project/pull/110238 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Concepts] Normalize SizeOfPackExpr's pack declaration (PR #110238)

2024-09-28 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 edited https://github.com/llvm/llvm-project/pull/110238 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] GH93099 (PR #110238)

2024-09-28 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 updated https://github.com/llvm/llvm-project/pull/110238 >From 23a765ac6d8e455121346405332d2066dcc0861e Mon Sep 17 00:00:00 2001 From: Younan Zhang Date: Fri, 27 Sep 2024 18:23:47 +0800 Subject: [PATCH 1/6] [Clang] GH93099 --- clang/include/clang/Sema/Sema.h

[clang] [Clang] GH93099 (PR #110238)

2024-09-28 Thread Younan Zhang via cfe-commits
zyn0217 wrote: The CI failure looks unrelated - it also *failed* in other PRs e.g. https://buildkite.com/llvm-project/github-pull-requests/builds/104957#019238e6-fd32-40a1-987a-4cd1c539926a (Which happens to fail in `std::common_type` that might have connections with the `sizeof...` expression

  1   2   >