[clang] [Clang][Sema] Revise the transformation of CTAD parameters of nested class templates (PR #91628)

https://github.com/zyn0217 updated https://github.com/llvm/llvm-project/pull/91628 >From 4b487ea33c33f53b5070f99965d0a9234ccb4138 Mon Sep 17 00:00:00 2001 From: Younan Zhang Date: Fri, 10 May 2024 01:09:29 +0800 Subject: [PATCH 1/2] [Clang][Sema] Revise the transformation of CTAD parameters of

[clang] [Clang] No longer require complete types with __builtin_launder (PR #91070)

MitalAshok wrote: Here's one scenario where it could make a difference in codegen : ```c++ // Header struct X; extern X x; void f(X&); inline void g() { f(x); } inline void h() { f(*__builtin_launder(__builtin_addressof(x))); } // Source struct X {

[clang] [Clang][Sema] Revise the transformation of CTAD parameters of nested class templates (PR #91628)

zyn0217 wrote: I'm merging it now - if something breaks and I'm not in front of my computer, feel free to revert then. https://github.com/llvm/llvm-project/pull/91628 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-b

[clang] 8c852ab - [Clang][Sema] Revise the transformation of CTAD parameters of nested class templates (#91628)

Author: Younan Zhang Date: 2024-05-10T20:47:15+08:00 New Revision: 8c852ab57932a5cd954cb0d050c3d2ab486428df URL: https://github.com/llvm/llvm-project/commit/8c852ab57932a5cd954cb0d050c3d2ab486428df DIFF: https://github.com/llvm/llvm-project/commit/8c852ab57932a5cd954cb0d050c3d2ab486428df.diff

[clang] [Clang][Sema] Revise the transformation of CTAD parameters of nested class templates (PR #91628)

https://github.com/zyn0217 closed https://github.com/llvm/llvm-project/pull/91628 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Catch missing format attributes (PR #70024)

https://github.com/budimirarandjelovicsyrmia updated https://github.com/llvm/llvm-project/pull/70024 From 024e4f0d0de64ba1f5f68eea578eb4ac1a554f0e Mon Sep 17 00:00:00 2001 From: budimirarandjelovicsyrmia Date: Fri, 5 Apr 2024 15:20:37 +0200 Subject: [PATCH] [clang] Catch missing format attribut

[clang] [clang] Catch missing format attributes (PR #70024)

budimirarandjelovicsyrmia wrote: ping https://github.com/llvm/llvm-project/pull/70024 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Enable C++17 relaxed template template argument matching by default (PR #89807)

alexfh wrote: Can we start with either reverting the whole commit or at least removing the flag deprecation and returning the old default (`-fno-relaxed-template-template-args`)? I think, @bgra8 and @sam-mccall can try to provide a reduced test case without renaming next week (it's sort of a

[clang] [clang][analyzer] Add checker 'security.SetgidSetuidOrder'. (PR #91445)

https://github.com/balazske updated https://github.com/llvm/llvm-project/pull/91445 From d839faf7a30851a172d812137b30635c741870f0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bal=C3=A1zs=20K=C3=A9ri?= Date: Wed, 8 May 2024 10:10:24 +0200 Subject: [PATCH 1/4] [clang][analyzer] Add checker 'Security

[clang] [clang] Enable C++17 relaxed template template argument matching by default (PR #89807)

Endilll wrote: > but not having a convenient reproducer is not a good reason to keep the ToT > Clang in a broken state As someone who worked on a different reduction of Sam's reproducer yesterday, and spent whopping 8 hours of work time to get https://github.com/llvm/llvm-project/issues/91566

[clang] [clang] Enable C++17 relaxed template template argument matching by default (PR #89807)

erichkeane wrote: > Can we start with either reverting the whole commit or at least removing the > flag deprecation and returning the old default > (`-fno-relaxed-template-template-args`)? I think, @bgra8 and @sam-mccall can > try to provide a reduced test case without renaming next week (it's

[clang] [Modules] Enable MS Windows test for implicit-module-no-timestamp.cpp (PR #91738)

https://github.com/ivanmurashko created https://github.com/llvm/llvm-project/pull/91738 There is a followup commit for #90319. The windows test was disabled at that commit but it should pass on MS Windows and it would be good to have it enabled. >From 6e5bd93173a8515749115f45c993ddf472437c79 M

[clang] [Modules] Enable MS Windows test for implicit-module-no-timestamp.cpp (PR #91738)

llvmbot wrote: @llvm/pr-subscribers-clang-modules @llvm/pr-subscribers-clang Author: Ivan Murashko (ivanmurashko) Changes There is a followup commit for #90319. The windows test was disabled at that commit but it should pass on MS Windows and it would be good to have it enabled. --- Ful

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

https://github.com/ymand commented: I think we can do this without resorting to a pointer union and templating of the initialization functions, but it will take a little refactoring. First, notice that `stmt` doesn't make sense for the CallStack. The call stack serves two purposes -- to avoid

[clang] [clang] Enable C++17 relaxed template template argument matching by default (PR #89807)

Endilll wrote: To be clear, we're asking for a reproducer with normal names first and foremost. I'm thankful for @bgra8 help running `creduce` over what they had, but we wouldn't had this particular conversation if they also provided full original reproducer that we could run `creduce` on with

[clang-tools-extra] 4cf3f03 - [clang-tidy] use llvm::any_of refactor getAnalyzerCheckersAndPackages [NFC] (#91713)

Author: Congcong Cai Date: 2024-05-10T21:51:07+08:00 New Revision: 4cf3f032283d8426c9b7829c7ccf0ab01939c7db URL: https://github.com/llvm/llvm-project/commit/4cf3f032283d8426c9b7829c7ccf0ab01939c7db DIFF: https://github.com/llvm/llvm-project/commit/4cf3f032283d8426c9b7829c7ccf0ab01939c7db.diff

[clang-tools-extra] [clang-tidy] use llvm::any_of refactor getAnalyzerCheckersAndPackages [NFC] (PR #91713)

https://github.com/HerrCai0907 closed https://github.com/llvm/llvm-project/pull/91713 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Fix incorrect passing of _BitInt args (PR #90741)

@@ -0,0 +1,14 @@ +// REQUIRES: arm-registered-target +// RUN: %clang_cc1 -triple aarch64-none-elf \ +// RUN: -O2 \ +// RUN: -emit-llvm -fexperimental-max-bitint-width=1024 -o - %s | FileCheck %s Lukacma wrote: I don't think such test file exists for aarch64.

[clang] 6317742 - [OpenACC][NFC] Fix isa behavior for OpenACC types

Author: erichkeane Date: 2024-05-10T07:03:48-07:00 New Revision: 63177422a834f4b81d59b827b5f2a1c5a9083749 URL: https://github.com/llvm/llvm-project/commit/63177422a834f4b81d59b827b5f2a1c5a9083749 DIFF: https://github.com/llvm/llvm-project/commit/63177422a834f4b81d59b827b5f2a1c5a9083749.diff LO

[clang] [z/OS] Set the default arch for z/OS to be arch10 (PR #89854)

perry-ca wrote: ping @MaskRay ^^^ https://github.com/llvm/llvm-project/pull/89854 ___ 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 expect no diagnosis test (PR #91293)

@@ -226,6 +236,10 @@ def run_clang_tidy(self): print("--") return clang_tidy_output +def check_no_diagnosis(self, clang_tidy_output): +if clang_tidy_output != "": +sys.exit

[clang-tools-extra] [clang-tidy] support expect no diagnosis test (PR #91293)

@@ -226,6 +236,10 @@ def run_clang_tidy(self): print("--") return clang_tidy_output +def check_no_diagnosis(self, clang_tidy_output): +if clang_tidy_output != "": +sys.exit

[clang] [llvm] [C++23] [CLANG] Adding C++23 constexpr math functions: fmin, fmax and frexp. (PR #88978)

https://github.com/zahiraam updated https://github.com/llvm/llvm-project/pull/88978 >From 3acc848f4fcc68445dfc849f9c6f8d384d3692af Mon Sep 17 00:00:00 2001 From: Zahira Ammarguellat Date: Tue, 16 Apr 2024 13:09:58 -0700 Subject: [PATCH 1/6] Adding C23 constexpr math functions fmin and frexp. -

[clang] [llvm] [AArch64] Add intrinsics for multi-vector to ZA array vector accumulators (PR #91606)

https://github.com/momchil-velikov updated https://github.com/llvm/llvm-project/pull/91606 >From 43fb20b7492307740c437e85c3f73af068d093cf Mon Sep 17 00:00:00 2001 From: Momchil Velikov Date: Thu, 9 May 2024 15:56:31 +0100 Subject: [PATCH] [AArch64] Add intrinsics for multi-vector to ZA array ve

[clang] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

https://github.com/hokein created https://github.com/llvm/llvm-project/pull/91750 None >From e7b23e02e9b4c98d9e24dd1debcdefada7a16004 Mon Sep 17 00:00:00 2001 From: Haojian Wu Date: Fri, 10 May 2024 16:31:20 +0200 Subject: [PATCH] [AST] Print the separator , for template arguments in ConceptR

[clang] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Haojian Wu (hokein) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/91750.diff 3 Files Affected: - (modified) clang/lib/AST/ASTConcept.cpp (+7-1) - (modified) clang/unittests/AST/CMakeLists.txt (+1) - (added) clang/u

[clang] [Clang][CWG1815] Support lifetime extension of temporary created by aggregate initialization using a default member initializer (PR #87933)

yronglin wrote: Many thanks for your review and help! I‘d like to take a few days to try to update static analyzer. https://github.com/llvm/llvm-project/pull/87933 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/

[clang] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

https://github.com/zyn0217 edited https://github.com/llvm/llvm-project/pull/91750 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

https://github.com/zyn0217 approved this pull request. Thanks! I'm surprised that this part was missing previously... https://github.com/llvm/llvm-project/pull/91750 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin

[clang] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

@@ -24,6 +24,7 @@ add_clang_unittest(ASTTests CommentLexer.cpp CommentParser.cpp CommentTextTest.cpp + ConceptPrinterTest.cpp zyn0217 wrote: Note: we probably want to update the gn build as well: https://github.com/llvm/llvm-project/tree/main/llvm/util

[clang] [clang] MangledSymbol: Added move to Names param in init list (PR #87287)

https://github.com/bradenhelmer updated https://github.com/llvm/llvm-project/pull/87287 >From 4d0290b2dc10705cf229e4c1c3b40dd5155ea61c Mon Sep 17 00:00:00 2001 From: Braden Helmer Date: Mon, 1 Apr 2024 18:09:59 -0400 Subject: [PATCH] Added move for Names parameter in MangledSymbol ctor --- cl

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

bazuzi wrote: I agree with the usefulness of storing the initial target separately, reducing checks and branching based on whether the current target is a `Stmt` or `Function`. Since I was touching everywhere `CallStack` was used anyway, I went ahead with not pushing the starting `FunctionDecl

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

https://github.com/bazuzi updated https://github.com/llvm/llvm-project/pull/91616 >From b62a95001e32a0c1d63204950481eb500c9d0275 Mon Sep 17 00:00:00 2001 From: Samira Bazuzi Date: Thu, 9 May 2024 12:00:06 -0400 Subject: [PATCH 1/3] [clang][dataflow] Fully support Environment construction for S

[clang] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

@@ -106,9 +106,15 @@ void ConceptReference::print(llvm::raw_ostream &OS, ConceptName.printName(OS, Policy); if (hasExplicitTemplateArgs()) { OS << "<"; +bool First = true; // FIXME: Find corresponding parameter for argument -for (auto &ArgLoc : ArgsAsWritte

[clang] [clang][analyzer] Add checker 'security.SetgidSetuidOrder'. (PR #91445)

@@ -0,0 +1,197 @@ +//===-- SetgidSetuidOrderChecker.cpp - check privilege revocation calls ---===// +// +// 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: Ap

[clang] [clang][analyzer] Add checker 'security.SetgidSetuidOrder'. (PR #91445)

@@ -0,0 +1,197 @@ +//===-- SetgidSetuidOrderChecker.cpp - check privilege revocation calls ---===// +// +// 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: Ap

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

ymand wrote: > I agree with the usefulness of storing the initial target separately, > reducing checks and branching based on whether the current target is a `Stmt` > or `Function`. Since I was touching everywhere `CallStack` was used anyway, I > went ahead with not pushing the starting `Funct

[clang-tools-extra] [clang-tidy] avoid false postive when ignore macro (PR #91757)

https://github.com/HerrCai0907 created https://github.com/llvm/llvm-project/pull/91757 Fixes: #91487 >From d340a04ffa50e03eb5267e267a27dcae5fefa223 Mon Sep 17 00:00:00 2001 From: Congcong Cai Date: Thu, 9 May 2024 00:51:58 +0800 Subject: [PATCH] [clang-tidy] avoid false postive when ignore ma

[clang-tools-extra] [clang-tidy] avoid false postive when ignore macro (PR #91757)

llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra Author: Congcong Cai (HerrCai0907) Changes Fixes: #91487 --- Full diff: https://github.com/llvm/llvm-project/pull/91757.diff 4 Files Affected: - (modified) clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp (+36

[clang-tools-extra] [clang-tidy] avoid false postive when ignore macro (PR #91757)

llvmbot wrote: @llvm/pr-subscribers-clang-tidy Author: Congcong Cai (HerrCai0907) Changes Fixes: #91487 --- Full diff: https://github.com/llvm/llvm-project/pull/91757.diff 4 Files Affected: - (modified) clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp (+36-25) -

[clang-tools-extra] [clang-tidy] `readability-simplify-boolean-expr` avoid to warn expression expand from macro when ``IgnoreMacro`` option is enabled. (PR #91757)

https://github.com/HerrCai0907 edited https://github.com/llvm/llvm-project/pull/91757 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] `readability-simplify-boolean-expr` avoid to warn expression expand from macro when ``IgnoreMacro`` option is enabled. (PR #91757)

https://github.com/HerrCai0907 updated https://github.com/llvm/llvm-project/pull/91757 >From e23675eae31ed9c3cc4bbf31d3bc43b492ef5eac Mon Sep 17 00:00:00 2001 From: Congcong Cai Date: Thu, 9 May 2024 00:51:58 +0800 Subject: [PATCH] [clang-tidy] avoid false postive when ignore macro Fixes: #914

[clang-tools-extra] [clang-tidy] support expect no diagnosis test (PR #91293)

@@ -226,6 +236,10 @@ def run_clang_tidy(self): print("--") return clang_tidy_output +def check_no_diagnosis(self, clang_tidy_output): +if clang_tidy_output != "": +sys.exit

[clang] [llvm] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

https://github.com/hokein updated https://github.com/llvm/llvm-project/pull/91750 >From e7b23e02e9b4c98d9e24dd1debcdefada7a16004 Mon Sep 17 00:00:00 2001 From: Haojian Wu Date: Fri, 10 May 2024 16:31:20 +0200 Subject: [PATCH 1/2] [AST] Print the separator , for template arguments in ConceptRef

[clang] [llvm] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

@@ -106,9 +106,15 @@ void ConceptReference::print(llvm::raw_ostream &OS, ConceptName.printName(OS, Policy); if (hasExplicitTemplateArgs()) { OS << "<"; +bool First = true; // FIXME: Find corresponding parameter for argument -for (auto &ArgLoc : ArgsAsWritte

[clang] [llvm] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

@@ -24,6 +24,7 @@ add_clang_unittest(ASTTests CommentLexer.cpp CommentParser.cpp CommentTextTest.cpp + ConceptPrinterTest.cpp hokein wrote: Done (we're not required to update gn build files, and there's a bot that can automate the porting). https://gi

[clang] [llvm] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

https://github.com/cor3ntin approved this pull request. https://github.com/llvm/llvm-project/pull/91750 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] d358b2d - [clang][NFC] Rename C++ CWG DR test files to use `cwg` prefix

Author: Vlad Serebrennikov Date: 2024-05-10T19:00:20+03:00 New Revision: d358b2de458e0611a3f474a9c3e3deff926c07c4 URL: https://github.com/llvm/llvm-project/commit/d358b2de458e0611a3f474a9c3e3deff926c07c4 DIFF: https://github.com/llvm/llvm-project/commit/d358b2de458e0611a3f474a9c3e3deff926c07c4.

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

https://github.com/bazuzi updated https://github.com/llvm/llvm-project/pull/91616 >From b62a95001e32a0c1d63204950481eb500c9d0275 Mon Sep 17 00:00:00 2001 From: Samira Bazuzi Date: Thu, 9 May 2024 12:00:06 -0400 Subject: [PATCH 1/4] [clang][dataflow] Fully support Environment construction for S

[clang] ef71c79 - [clang][NFC] Update C++ DR status page

Author: Vlad Serebrennikov Date: 2024-05-10T19:03:21+03:00 New Revision: ef71c79d85fb3f94f19a92a84ab44719d112c1bd URL: https://github.com/llvm/llvm-project/commit/ef71c79d85fb3f94f19a92a84ab44719d112c1bd DIFF: https://github.com/llvm/llvm-project/commit/ef71c79d85fb3f94f19a92a84ab44719d112c1bd.

[clang] [clang][analyzer] Add checker 'security.SetgidSetuidOrder'. (PR #91445)

https://github.com/balazske deleted https://github.com/llvm/llvm-project/pull/91445 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][analyzer] Add checker 'security.SetgidSetuidOrder'. (PR #91445)

balazske wrote: Is it useful to add a note tag to the previous `setuid(getuid())` call? It can be (theoretically) in another function or otherwise in a remote place in the source code. https://github.com/llvm/llvm-project/pull/91445 ___ cfe-commits m

[clang] [clang][analyzer] Add checker 'security.SetgidSetuidOrder'. (PR #91445)

https://github.com/balazske updated https://github.com/llvm/llvm-project/pull/91445 From d839faf7a30851a172d812137b30635c741870f0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bal=C3=A1zs=20K=C3=A9ri?= Date: Wed, 8 May 2024 10:10:24 +0200 Subject: [PATCH 1/5] [clang][analyzer] Add checker 'Security

[clang] 099c152 - [AST] Print the separator "," for template arguments in ConceptReference::print (#91750)

Author: Haojian Wu Date: 2024-05-10T18:13:47+02:00 New Revision: 099c1527f6088305fc253e9598685faa85544a8e URL: https://github.com/llvm/llvm-project/commit/099c1527f6088305fc253e9598685faa85544a8e DIFF: https://github.com/llvm/llvm-project/commit/099c1527f6088305fc253e9598685faa85544a8e.diff LO

[clang] [llvm] [AST] Print the separator "," for template arguments in ConceptReference::print (PR #91750)

https://github.com/hokein closed https://github.com/llvm/llvm-project/pull/91750 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][analyzer] Add checker 'security.SetgidSetuidOrder'. (PR #91445)

@@ -0,0 +1,170 @@ +// RUN: %clang_analyze_cc1 -analyzer-checker=core,security.SetgidSetuidOrder -verify %s + +#include "Inputs/system-header-simulator-setgid-setuid.h" + +void correct_order() { + if (setgid(getgid()) == -1) +return; + if (setuid(getuid()) == -1) +return

[clang] Re-apply "Emit missing cleanups for stmt-expr" and other commits (PR #89154)

higher-performance wrote: @usx95 I think there may still be a bug (unless the trunk builds I'm trying aren't, in fact, the trunk). See: #91761 https://github.com/llvm/llvm-project/pull/89154 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http

[clang] [clang][analyzer] Add checker 'security.SetgidSetuidOrder'. (PR #91445)

@@ -0,0 +1,197 @@ +//===-- SetgidSetuidOrderChecker.cpp - check privilege revocation calls ---===// +// +// 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: Ap

[clang] [llvm] [OpenMP] Remove dependency on `libffi` from offloading runtime (PR #91264)

https://github.com/jhuber6 edited https://github.com/llvm/llvm-project/pull/91264 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [InstallAPI] Support mutually exclusive parse options (PR #90686)

https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/90686 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 062f6fe - [InstallAPI] Support mutually exclusive parse options (#90686)

Author: Cyndy Ishida Date: 2024-05-10T09:35:41-07:00 New Revision: 062f6fe324e98b0994e49bc14eb45b20aa0807c4 URL: https://github.com/llvm/llvm-project/commit/062f6fe324e98b0994e49bc14eb45b20aa0807c4 DIFF: https://github.com/llvm/llvm-project/commit/062f6fe324e98b0994e49bc14eb45b20aa0807c4.diff

[clang] [clang] Add test for CWG1820 "Qualified typedef names" (PR #91765)

https://github.com/Endilll created https://github.com/llvm/llvm-project/pull/91765 [P1787R6](https://wg21.link/p1787r6): > [CWG1820](https://cplusplus.github.io/CWG/issues/1820.html) is resolved by > requiring that a qualified declarator-id declare an entity. P1787R6 changes wording of [dcl.pr

[clang] [clang] Add test for CWG1820 "Qualified typedef names" (PR #91765)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Vlad Serebrennikov (Endilll) Changes [P1787R6](https://wg21.link/p1787r6): > [CWG1820](https://cplusplus.github.io/CWG/issues/1820.html) is resolved by requiring that a qualified declarator-id declare an entity. P1787R6 changes wording of

[clang-tools-extra] [clang-tidy] `readability-simplify-boolean-expr` avoid to warn expression expand from macro when ``IgnoreMacro`` option is enabled. (PR #91757)

@@ -357,6 +357,9 @@ Changes in existing checks support calls to overloaded operators as base expression and provide fixes to expressions with side-effects. +- Improved :doc:`readability-simplify-boolean-expr` PiotrZSL wrote: split into 2 lines https://g

[clang-tools-extra] [clang-tidy] `readability-simplify-boolean-expr` avoid to warn expression expand from macro when ``IgnoreMacro`` option is enabled. (PR #91757)

https://github.com/PiotrZSL edited https://github.com/llvm/llvm-project/pull/91757 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] `readability-simplify-boolean-expr` avoid to warn expression expand from macro when ``IgnoreMacro`` option is enabled. (PR #91757)

https://github.com/PiotrZSL approved this pull request. Just nits. https://github.com/llvm/llvm-project/pull/91757 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] `readability-simplify-boolean-expr` avoid to warn expression expand from macro when ``IgnoreMacro`` option is enabled. (PR #91757)

@@ -513,17 +513,25 @@ class SimplifyBooleanExprCheck::Visitor : public RecursiveASTVisitor { return true; } - static bool isUnaryLNot(const Expr *E) { -return isa(E) && + static bool isExpectedUnaryLNot(SimplifyBooleanExprCheck *Check, +

[clang] [clang] Enable C++17 relaxed template template argument matching by default (PR #89807)

mizvekov wrote: I am repeating myself here, but the crash happens after a bunch of errors: it's not significant, we have evidence this sort of crash is associated with error recovery. This patch implements a standard mandated breaking change, and this 'stdexec' is user code. Without evidence

[clang] [clang] Enable C++17 relaxed template template argument matching by default (PR #89807)

erichkeane wrote: >I am repeating myself here, but the crash happens after a bunch of errors: >it's not significant, we have evidence this sort of crash is associated with >error recovery. I missed this, thanks for clarifying (godbolt link scrolled and all I saw was warnings!). A crash-on-in

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

https://github.com/bazuzi updated https://github.com/llvm/llvm-project/pull/91616 >From b62a95001e32a0c1d63204950481eb500c9d0275 Mon Sep 17 00:00:00 2001 From: Samira Bazuzi Date: Thu, 9 May 2024 12:00:06 -0400 Subject: [PATCH 1/5] [clang][dataflow] Fully support Environment construction for S

[clang] [clang] CTAD alias: fix the transformation for the require-clause expr (PR #90961)

https://github.com/hokein edited https://github.com/llvm/llvm-project/pull/90961 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Enable C++17 relaxed template template argument matching by default (PR #89807)

mizvekov wrote: By the way, creduce/cvise won't help much here unless the interestingness test accounts for 'works on GCC'. Otherwise, It'd be trivial to conjure some snippet of code that works before P0522, but breaks afterward as intended. https://github.com/llvm/llvm-project/pull/89807 ___

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

https://github.com/ymand edited https://github.com/llvm/llvm-project/pull/91616 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

@@ -718,29 +730,46 @@ class Environment { void pushCallInternal(const FunctionDecl *FuncDecl, ArrayRef Args); + // FIXME: Add support for resetting globals after function calls to enable + // the implementation of sound analyses. + /// Assigns st

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

https://github.com/ymand approved this pull request. Nice! https://github.com/llvm/llvm-project/pull/91616 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

@@ -476,7 +476,7 @@ runTypeErasedDataflowAnalysis( PrettyStackTraceAnalysis CrashInfo(ACFG, "runTypeErasedDataflowAnalysis"); std::optional MaybeStartingEnv; - if (InitEnv.callStackSize() == 1) { + if (InitEnv.callStackSize() == 0) { ymand wrote: aside:

[clang] [clang][dataflow] Fully support Environment construction for Stmt analysis. (PR #91616)

https://github.com/bazuzi updated https://github.com/llvm/llvm-project/pull/91616 >From b62a95001e32a0c1d63204950481eb500c9d0275 Mon Sep 17 00:00:00 2001 From: Samira Bazuzi Date: Thu, 9 May 2024 12:00:06 -0400 Subject: [PATCH 1/6] [clang][dataflow] Fully support Environment construction for S

[clang] [InstallAPI] add JSON option to pass X arguments (PR #91770)

https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/91770 None >From 480f1a6dd0c78c8018ac08259b4d3cba64c25165 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 2 May 2024 19:53:07 -0700 Subject: [PATCH] [InstallAPI] add JSON option to pass X arguments --- ..

[clang] [InstallAPI] add JSON option to pass X arguments (PR #91770)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Cyndy Ishida (cyndyishida) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/91770.diff 9 Files Affected: - (modified) clang/include/clang/Basic/DiagnosticInstallAPIKinds.td (+1-1) - (modified) clang/test/InstallAPI/ali

[clang] [lld] [llvm] [RISCV] Support .note.gnu.property for enable Zicfiss and Zicfilp extension (PR #77414)

@@ -5272,8 +5272,16 @@ static std::string getGNUProperty(uint32_t Type, uint32_t DataSize, return OS.str(); case GNU_PROPERTY_AARCH64_FEATURE_1_AND: case GNU_PROPERTY_X86_FEATURE_1_AND: -OS << ((Type == GNU_PROPERTY_AARCH64_FEATURE_1_AND) ? "aarch64 feature: " -

[clang] [lld] [llvm] [RISCV] Support .note.gnu.property for enable Zicfiss and Zicfilp extension (PR #77414)

@@ -468,6 +468,13 @@ static void checkOptions() { error("-z pauth-report only supported on AArch64"); } + if (config->emachine != EM_RISCV) { +if (config->zZicfilpReport != "none") + error("-z zicfilip-report only support on RISCV32/RISCV64"); ---

[clang] [Driver] Clean up fp-contract handling in clang driver (PR #91271)

@@ -238,3 +238,17 @@ // RUN: %clang -### -fno-unsafe-math-optimizations -funsafe-math-optimizations \ // RUN: -ffp-contract=off -c %s 2>&1 | FileCheck --check-prefix=CHECK-FPC-OFF %s +// RUN: %clang -### -funsafe-math-optimizations -ffp-contract=off -c %s 2>&1 \

[clang] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors (PR #91777)

https://github.com/danakj created https://github.com/llvm/llvm-project/pull/91777 The -Wunsafe-buffer-usage warning should fire on any call to a function annotated with [[clang::unsafe_buffer_usage]], however it omitted calls to constructors, since the expression is a CXXConstructExpr which do

[clang] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors (PR #91777)

danakj wrote: cc: @haoNoQ https://github.com/llvm/llvm-project/pull/91777 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors (PR #91777)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Dana Jansens (danakj) Changes The -Wunsafe-buffer-usage warning should fire on any call to a function annotated with [[clang::unsafe_buffer_usage]], however it omitted calls to constructors, since the expression is a CXXConstructExpr whic

[clang] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors (PR #91777)

llvmbot wrote: @llvm/pr-subscribers-clang-analysis Author: Dana Jansens (danakj) Changes The -Wunsafe-buffer-usage warning should fire on any call to a function annotated with [[clang::unsafe_buffer_usage]], however it omitted calls to constructors, since the expression is a CXXConstruct

[clang] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors (PR #91777)

https://github.com/danakj updated https://github.com/llvm/llvm-project/pull/91777 >From 1c835f4cc77823360762b8e53e2bbfe027c977cf Mon Sep 17 00:00:00 2001 From: danakj Date: Fri, 10 May 2024 13:31:17 -0400 Subject: [PATCH] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors T

[clang] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors (PR #91777)

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 317e6ff6290c4c3065cb79c3eaf52f171e40cdfd d7694d70393fd49792470147c7ca136fd33f194e --

[clang] [Driver] Clean up fp-contract handling in clang driver (PR #91271)

@@ -2751,6 +2751,7 @@ static void RenderFloatingPointOptions(const ToolChain &TC, const Driver &D, // If one wasn't given by the user, don't pass it here. StringRef FPContract; StringRef LastSeenFfpContractOption; + StringRef LastFpContractOverrideOption; --

[clang] Add support for builtin_verbose_trap (PR #79230)

@@ -172,6 +172,27 @@ static bool checkArgCount(Sema &S, CallExpr *Call, unsigned DesiredArgCount) { << /*is non object*/ 0 << Call->getArg(1)->getSourceRange(); } +static bool checkBuiltinVerboseTrap(CallExpr *Call, Sema &S) { delcypher wrote: @ahat

[clang] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors (PR #91777)

https://github.com/danakj updated https://github.com/llvm/llvm-project/pull/91777 >From 8b318dadac6d0ec53b5d26461edfe19a391845ec Mon Sep 17 00:00:00 2001 From: danakj Date: Fri, 10 May 2024 13:31:17 -0400 Subject: [PATCH] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors T

[clang] 514d80b - [Driver, test] Add -Werror to -ffp-contract/-ffp-model tests

Author: Fangrui Song Date: 2024-05-10T10:49:52-07:00 New Revision: 514d80b4feea3c788c1b0821959e96f63c8b8f3d URL: https://github.com/llvm/llvm-project/commit/514d80b4feea3c788c1b0821959e96f63c8b8f3d DIFF: https://github.com/llvm/llvm-project/commit/514d80b4feea3c788c1b0821959e96f63c8b8f3d.diff

[clang] [lld] [llvm] [RISCV] Support .note.gnu.property for enable Zicfiss and Zicfilp extension (PR #77414)

MaskRay wrote: Unless `lld/test/ELF` tests will get a failure, the `lld/` part should ideally be dropped from this PR. lld folks (like I) might not notice this patch if there is just a `[RISCV]` tag. https://github.com/llvm/llvm-project/pull/77414 ___

[clang] [Driver] Clean up fp-contract handling in clang driver (PR #91271)

@@ -238,3 +238,17 @@ // RUN: %clang -### -fno-unsafe-math-optimizations -funsafe-math-optimizations \ // RUN: -ffp-contract=off -c %s 2>&1 | FileCheck --check-prefix=CHECK-FPC-OFF %s +// RUN: %clang -### -funsafe-math-optimizations -ffp-contract=off -c %s 2>&1 \

[clang] Respect the [[clang::unsafe_buffer_usage]] attribute for constructors (PR #91777)

https://github.com/danakj updated https://github.com/llvm/llvm-project/pull/91777 >From 8b318dadac6d0ec53b5d26461edfe19a391845ec Mon Sep 17 00:00:00 2001 From: danakj Date: Fri, 10 May 2024 13:31:17 -0400 Subject: [PATCH 1/2] Respect the [[clang::unsafe_buffer_usage]] attribute for constructor

[clang] [clang][NFC] Remove class layout scissor (PR #89055)

https://github.com/urnathan updated https://github.com/llvm/llvm-project/pull/89055 >From 9ab483f3451bfcaa7968c5f1cf7115144522f58a Mon Sep 17 00:00:00 2001 From: Nathan Sidwell Date: Mon, 1 Apr 2024 16:15:12 -0400 Subject: [PATCH 1/3] [clang] Remove class layout scissor --- clang/lib/CodeGen/

[clang] [lld] [llvm] [RISCV] Support .note.gnu.property for enable Zicfiss and Zicfilp extension (PR #77414)

@@ -0,0 +1,29 @@ +// When -march with zicfiss0p4 or zicfilp0p4 add GNU property to file object MaskRay wrote: This directory is for driver tests. Codegen tests should use `%clang_cc1` and be moved elsewhere (e.g. test/CodeGen) https://github.com/llvm/llvm-proje

[clang] [test] Move RISCV tests to clang/test/CodeGen/RISCV/ (PR #91783)

https://github.com/MaskRay created https://github.com/llvm/llvm-project/pull/91783 None >From 79c0d400fee943f794ed38965cec0c8b0d2dffef Mon Sep 17 00:00:00 2001 From: Fangrui Song Date: Fri, 10 May 2024 11:00:14 -0700 Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=

[clang] Enable unguarded availability diagnostic on instantiated template functions (PR #91699)

https://github.com/hekota edited https://github.com/llvm/llvm-project/pull/91699 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [test] Move RISCV tests to clang/test/CodeGen/RISCV/ (PR #91783)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Fangrui Song (MaskRay) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/91783.diff 13 Files Affected: - (renamed) clang/test/CodeGen/RISCV/attr-rvv-vector-bits-bitcast.c () - (renamed) clang/test/CodeGen/RISCV/attr-rvv

<    1   2   3   4   >