[clang] 947b9f5 - [clang-repl] Fix printing preprocessed tokens and macros (#104964)

2024-08-21 Thread via cfe-commits
Author: Jonas Hahnfeld Date: 2024-08-21T09:09:06+02:00 New Revision: 947b9f55b5f327e14368a48fb6ce10242ea29bf3 URL: https://github.com/llvm/llvm-project/commit/947b9f55b5f327e14368a48fb6ce10242ea29bf3 DIFF: https://github.com/llvm/llvm-project/commit/947b9f55b5f327e14368a48fb6ce10242ea29bf3.diff

[clang] [clang-repl] Fix printing preprocessed tokens and macros (PR #104964)

2024-08-21 Thread Jonas Hahnfeld via cfe-commits
https://github.com/hahnjo closed https://github.com/llvm/llvm-project/pull/104964 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [RISCV] Remove experimental for Ssqosid ext (PR #105476)

2024-08-21 Thread Shao-Ce SUN via cfe-commits
https://github.com/sunshaoce created https://github.com/llvm/llvm-project/pull/105476 Ratified: https://github.com/riscv/riscv-ssqosid/releases/tag/v1.0 >From acffcc86ffd145849ffeb884ab31d32c5cff5710 Mon Sep 17 00:00:00 2001 From: Shao-Ce SUN Date: Wed, 21 Aug 2024 15:12:32 +0800 Subject: [PAT

[clang] [llvm] [RISCV] Remove experimental for Ssqosid ext (PR #105476)

2024-08-21 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-backend-risc-v Author: Shao-Ce SUN (sunshaoce) Changes Ratified: https://github.com/riscv/riscv-ssqosid/releases/tag/v1.0 --- Full diff: https://github.com/llvm/llvm-project/pull/105476.diff 6 Files Affected: - (modified) clang/test/Driver/print-supp

[clang] [llvm] [RISCV] Remove experimental for Ssqosid ext (PR #105476)

2024-08-21 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Shao-Ce SUN (sunshaoce) Changes Ratified: https://github.com/riscv/riscv-ssqosid/releases/tag/v1.0 --- Full diff: https://github.com/llvm/llvm-project/pull/105476.diff 6 Files Affected: - (modified) clang/test/Driver/print-supported-ext

RE: cfe-commits Digest, Vol 206, Issue 1693

2024-08-21 Thread via cfe-commits
Questioned  1 of 10 FRM:cfe-commits-bounces@lists.llvm.org SU &  -Original Message- From: Sent: Wed, 21 Aug 2024 00:14:58 -0700 To: 7578703...@txt.att.net Subject: =?utf-8?q?cfe-commits_Digest=2C_Vol_206=2C_Issue_169

[clang] [llvm] [X86][AVX10.2] Support AVX10.2-CONVERT new instructions. (PR #101600)

2024-08-21 Thread Freddy Ye via cfe-commits
https://github.com/FreddyLeaf closed https://github.com/llvm/llvm-project/pull/101600 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 9fa2386 - [RISCV] Add Hazard3 Core as taped out for RP2350 (#102452)

2024-08-21 Thread via cfe-commits
Author: Sam Elliott Date: 2024-08-21T08:45:45+01:00 New Revision: 9fa2386ff13289d46ebf31656f4be7859f501468 URL: https://github.com/llvm/llvm-project/commit/9fa2386ff13289d46ebf31656f4be7859f501468 DIFF: https://github.com/llvm/llvm-project/commit/9fa2386ff13289d46ebf31656f4be7859f501468.diff L

[clang] [llvm] [RISCV] Add Hazard3 Core as taped out for RP2350 (PR #102452)

2024-08-21 Thread Sam Elliott via cfe-commits
https://github.com/lenary closed https://github.com/llvm/llvm-project/pull/102452 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [RISCV] Remove experimental for Ssqosid ext (PR #105476)

2024-08-21 Thread Yingwei Zheng via cfe-commits
https://github.com/dtcxzyw approved this pull request. LGTM. https://github.com/llvm/llvm-project/pull/105476 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][ASTImporter] New fix for default template parameter values. (PR #101836)

2024-08-21 Thread Balázs Kéri via cfe-commits
@@ -5968,11 +5962,21 @@ ASTNodeImporter::VisitTemplateTypeParmDecl(TemplateTypeParmDecl *D) { } if (D->hasDefaultArgument()) { +// Default argument can be "inherited" when it has a reference to the +// previous declaration (of the default argument) which is stored

[clang] [PS5][Driver] Link main components with -pie by default (PR #102901)

2024-08-21 Thread Edd Dawson via cfe-commits
https://github.com/playstation-edd updated https://github.com/llvm/llvm-project/pull/102901 >From 0ca2239d21cb414ce5e6fb9057d442c18f61a32e Mon Sep 17 00:00:00 2001 From: Edd Dawson Date: Mon, 12 Aug 2024 14:11:11 +0100 Subject: [PATCH 1/3] [PS5][Driver] Link main components with -pie by default

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

2024-08-21 Thread Budimir Aranđelović via cfe-commits
https://github.com/budimirarandjelovichtec created https://github.com/llvm/llvm-project/pull/105479 Enable flag -Wmissing-format-attribute to catch missing attributes. Fixes https://github.com/llvm/llvm-project/issues/60718 From 69f6b14f71222d66bb7785fef51514431e86d1e4 Mon Sep 17 00:00:00 2001

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

2024-08-21 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Budimir Aranđelović (budimirarandjelovichtec) Changes Enable flag -Wmissing-format-attribute to catch missing attributes. Fixes https://github.com/llvm/llvm-project/issues/60718 --- Patch is 40.79 KiB, truncated to 20.00 KiB below, full

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

2024-08-21 Thread Budimir Aranđelović via cfe-commits
budimirarandjelovichtec wrote: @AaronBallman @aaronpuchert @vitalybuka @Endilll Last found issues are addressed and new PR is created: https://github.com/llvm/llvm-project/pull/105479 https://github.com/llvm/llvm-project/pull/70024 ___ cfe-commits ma

[clang] 9d739e5 - [Clang] Implement CWG2351 `void{}` (#78060)

2024-08-21 Thread via cfe-commits
Author: Mital Ashok Date: 2024-08-21T10:09:08+02:00 New Revision: 9d739e54f4506bf9bd220c5d65f710b86a39f6d5 URL: https://github.com/llvm/llvm-project/commit/9d739e54f4506bf9bd220c5d65f710b86a39f6d5 DIFF: https://github.com/llvm/llvm-project/commit/9d739e54f4506bf9bd220c5d65f710b86a39f6d5.diff L

[clang] [Clang] Implement CWG2351 `void{}` (PR #78060)

2024-08-21 Thread via cfe-commits
https://github.com/cor3ntin closed https://github.com/llvm/llvm-project/pull/78060 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Support -Wa, options -mmsa and -mno-msa (PR #99615)

2024-08-21 Thread via cfe-commits
@@ -0,0 +1,8 @@ +// RUN: %clang -### -c --target=mips64el-unknown-linux-gnuabi64 \ yingopq wrote: OK,Thanks! https://github.com/llvm/llvm-project/pull/99615 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https

[clang] [clang][ASTImporter] New fix for default template parameter values. (PR #101836)

2024-08-21 Thread Balázs Kéri via cfe-commits
@@ -5968,11 +5962,21 @@ ASTNodeImporter::VisitTemplateTypeParmDecl(TemplateTypeParmDecl *D) { } if (D->hasDefaultArgument()) { +// Default argument can be "inherited" when it has a reference to the +// previous declaration (of the default argument) which is stored

[clang] [Clang] [NFC] Introduce `DynamicRecursiveASTVisitor` (PR #105195)

2024-08-21 Thread via cfe-commits
cor3ntin wrote: This is a very impressive piece of work! I think we are looking at a non-trivial choice: - Do we make clang faster to compile for ourselves, or slightly faster for everyone? My initial thinking is that clang is run orders of magnitude more often than it is compile, and so impr

[clang] [clang] check deduction consistency when partial ordering function templates (PR #100692)

2024-08-21 Thread via cfe-commits
@@ -5346,7 +5346,6 @@ static EvalStmtResult EvaluateStmt(StmtResult &Result, EvalInfo &Info, const Expr *RetExpr = cast(S)->getRetValue(); FullExpressionRAII Scope(Info); if (RetExpr && RetExpr->isValueDependent()) { - EvaluateDependentExpr(RetExpr, Info); ---

[clang] [clang-format] Don't insert a space between :: and * (PR #105043)

2024-08-21 Thread via cfe-commits
https://github.com/mydeveloperday approved this pull request. https://github.com/llvm/llvm-project/pull/105043 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 126b56a - [RISCV] Make EmitRISCVCpuSupports accept multiple features (#104917)

2024-08-21 Thread via cfe-commits
Author: Piyou Chen Date: 2024-08-21T16:46:59+08:00 New Revision: 126b56a234486a2cd05a8beca78bcf89fe47d167 URL: https://github.com/llvm/llvm-project/commit/126b56a234486a2cd05a8beca78bcf89fe47d167 DIFF: https://github.com/llvm/llvm-project/commit/126b56a234486a2cd05a8beca78bcf89fe47d167.diff LO

[clang] [llvm] [RISC-V] Make EmitRISCVCpuSupports accept multiple features (PR #104917)

2024-08-21 Thread Piyou Chen via cfe-commits
https://github.com/BeMg closed https://github.com/llvm/llvm-project/pull/104917 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Concepts] Fix the constraint equivalence checking for TemplateTypeParmTypes (PR #102131)

2024-08-21 Thread via cfe-commits
cor3ntin wrote: What's the status of that? https://github.com/llvm/llvm-project/pull/102131 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][NFCI] Cleanup the fix for default function argument substitution (PR #104911)

2024-08-21 Thread Younan Zhang via cfe-commits
@@ -285,7 +286,8 @@ Response HandleFunction(Sema &SemaRef, const FunctionDecl *Function, // If this function was instantiated from a specialized member that is // a function template, we're done. assert(Function->getPrimaryTemplate() && "No function template?"); -

[clang] 8ac140f - [Clang][NFCI] Cleanup the fix for default function argument substitution (#104911)

2024-08-21 Thread via cfe-commits
Author: Younan Zhang Date: 2024-08-21T17:01:05+08:00 New Revision: 8ac140f390847e4e85e0a4fd910baaf46e5d115b URL: https://github.com/llvm/llvm-project/commit/8ac140f390847e4e85e0a4fd910baaf46e5d115b DIFF: https://github.com/llvm/llvm-project/commit/8ac140f390847e4e85e0a4fd910baaf46e5d115b.diff

[clang] [Clang][NFCI] Cleanup the fix for default function argument substitution (PR #104911)

2024-08-21 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 closed https://github.com/llvm/llvm-project/pull/104911 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] fix generic lambda inside requires-clause of friend function template (PR #99813)

2024-08-21 Thread via cfe-commits
https://github.com/cor3ntin updated https://github.com/llvm/llvm-project/pull/99813 >From 5d7f291e35930e07f52a7ac17a09f93690b64def Mon Sep 17 00:00:00 2001 From: Backl1ght Date: Sun, 21 Jul 2024 23:43:24 +0800 Subject: [PATCH 1/2] fix --- clang/docs/ReleaseNotes.rst | 2 ++

[clang] [Clang][Concepts] Fix the constraint equivalence checking for TemplateTypeParmTypes (PR #102131)

2024-08-21 Thread Younan Zhang via cfe-commits
zyn0217 wrote: @mizvekov suggested in https://github.com/llvm/llvm-project/issues/101735#issuecomment-2273950029 we should try to decouple the constraint checking from the redeclaration chain building, however that is somehow involved. https://github.com/llvm/llvm-project/pull/102131

[clang] [clang] Diagnose functions with too many parameters (PR #104833)

2024-08-21 Thread via cfe-commits
cor3ntin wrote: @Endilll you forgot to update the tests - hence the ci failure https://github.com/llvm/llvm-project/pull/104833 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] [NFC] Introduce `DynamicRecursiveASTVisitor` (PR #105195)

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

[clang] [Clang] [NFC] Introduce `DynamicRecursiveASTVisitor` (PR #105195)

2024-08-21 Thread Younan Zhang via cfe-commits
@@ -0,0 +1,252 @@ +#ifndef LLVM_CLANG_AST_DYNAMIC_RECURSIVE_AST_VISITOR_H zyn0217 wrote: Don't forget the license header :) https://github.com/llvm/llvm-project/pull/105195 ___ cfe-commits mailing list cfe-commits@list

[clang] [Clang] [NFC] Introduce `DynamicRecursiveASTVisitor` (PR #105195)

2024-08-21 Thread Younan Zhang via cfe-commits
@@ -0,0 +1,252 @@ +#ifndef LLVM_CLANG_AST_DYNAMIC_RECURSIVE_AST_VISITOR_H +#define LLVM_CLANG_AST_DYNAMIC_RECURSIVE_AST_VISITOR_H + +#include "clang/AST/Attr.h" +#include "clang/AST/ExprConcepts.h" +#include "clang/AST/TypeLoc.h" + +namespace clang { +class ASTContext; + +/// Recu

[clang] [Clang] [NFC] Introduce `DynamicRecursiveASTVisitor` (PR #105195)

2024-08-21 Thread Younan Zhang via cfe-commits
https://github.com/zyn0217 commented: Thanks! https://github.com/llvm/llvm-project/pull/105195 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [RISCV][FMV] Support target_clones (PR #85786)

2024-08-21 Thread Piyou Chen via cfe-commits
https://github.com/BeMg updated https://github.com/llvm/llvm-project/pull/85786 >From 395ce72afbf9e4b12fcbfaf9cdbda8921c9ff72a Mon Sep 17 00:00:00 2001 From: Piyou Chen Date: Tue, 23 Jul 2024 19:59:06 -0700 Subject: [PATCH 1/2] [RISCV][FMV] Support target_clones --- .../clang/Basic/DiagnosticF

[clang] [PS5][Driver] Link main components with -pie by default (PR #102901)

2024-08-21 Thread Edd Dawson via cfe-commits
@@ -237,7 +237,10 @@ void tools::PS5cpu::Linker::ConstructJob(Compilation &C, const JobAction &JA, if (!D.SysRoot.empty()) CmdArgs.push_back(Args.MakeArgString("--sysroot=" + D.SysRoot)); - if (Args.hasArg(options::OPT_pie)) + // Default to PIE for non-static main com

[clang] [PS5][Driver] Link main components with -pie by default (PR #102901)

2024-08-21 Thread Edd Dawson via cfe-commits
@@ -1,3 +1,18 @@ +// Test that PIE is the default for main components + +// RUN: %clang --target=x86_64-scei-ps5 %s -### 2>&1 | FileCheck --check-prefixes=CHECK-PIE %s + +// CHECK-PIE: {{ld(\.exe)?}}" +// CHECK-PIE-SAME: "-pie" + +// RUN: %clang --target=x86_64-scei-ps5 -no-pie %

[clang] [libclang/python] Do not rely on `ctypes`' `errcheck` (PR #105490)

2024-08-21 Thread Jannick Kremer via cfe-commits
https://github.com/DeinAlptraum created https://github.com/llvm/llvm-project/pull/105490 Call conversion functions directly instead of using them for type conversion on library function calls via `ctypes`' `errcheck` functionality. >From 8cf94c6486b47cbe4f9ec82cc7c8a9c4948ed012 Mon Sep 17 00:0

[clang] [libclang/python] Do not rely on `ctypes`' `errcheck` (PR #105490)

2024-08-21 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Jannick Kremer (DeinAlptraum) Changes Call conversion functions directly instead of using them for type conversion on library function calls via `ctypes`' `errcheck` functionality. --- Patch is 35.75 KiB, truncated to 20.00 KiB below, fu

[clang] 297bb46 - [PS5][Driver] Link main components with -pie by default (#102901)

2024-08-21 Thread via cfe-commits
Author: Edd Dawson Date: 2024-08-21T10:53:45+01:00 New Revision: 297bb467acd31447d64f0540835127d50408e87d URL: https://github.com/llvm/llvm-project/commit/297bb467acd31447d64f0540835127d50408e87d DIFF: https://github.com/llvm/llvm-project/commit/297bb467acd31447d64f0540835127d50408e87d.diff LO

[clang] [PS5][Driver] Link main components with -pie by default (PR #102901)

2024-08-21 Thread Edd Dawson via cfe-commits
https://github.com/playstation-edd closed https://github.com/llvm/llvm-project/pull/102901 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][ASTImporter] set nonnull type for var specialized from lambda template (PR #105492)

2024-08-21 Thread Ding Fei via cfe-commits
https://github.com/danix800 created https://github.com/llvm/llvm-project/pull/105492 There's circular deps here when importing btw VarTemplateSpecializationDecl (m) and its type. VarDecl is created with a null type QualType(), and the importing of its type will leads to linkage computation (f

[clang] [clang][ASTImporter] set nonnull type for var specialized from lambda template (PR #105492)

2024-08-21 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Ding Fei (danix800) Changes There's circular deps here when importing btw VarTemplateSpecializationDecl (m) and its type. VarDecl is created with a null type QualType(), and the importing of its type will leads to linkage computation (fo

[clang] [libclang/python] Do not rely on `ctypes`' `errcheck` (PR #105490)

2024-08-21 Thread Jannick Kremer via cfe-commits
https://github.com/DeinAlptraum edited https://github.com/llvm/llvm-project/pull/105490 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [libclang/python] Do not rely on `ctypes`' `errcheck` (PR #105490)

2024-08-21 Thread Jannick Kremer via cfe-commits
https://github.com/DeinAlptraum commented: @Endilll this removes the `errcheck` magic conversion stuff that we discussed the other day https://github.com/llvm/llvm-project/pull/105490 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lis

[clang] [libclang/python] Do not rely on `ctypes`' `errcheck` (PR #105490)

2024-08-21 Thread Jannick Kremer via cfe-commits
@@ -424,7 +424,7 @@ def location(self): @property def spelling(self): -return conf.lib.clang_getDiagnosticSpelling(self) # type: ignore [no-any-return] +return _CXString.from_result(conf.lib.clang_getDiagnosticSpelling(self)) DeinAlp

[clang] [libclang/python] Do not rely on `ctypes`' `errcheck` (PR #105490)

2024-08-21 Thread Jannick Kremer via cfe-commits
@@ -3743,82 +3765,62 @@ def write_main_file_to_stdout(self): ("clang_equalLocations", [SourceLocation, SourceLocation], bool), ("clang_equalRanges", [SourceRange, SourceRange], bool), ("clang_equalTypes", [Type, Type], bool), -("clang_formatDiagnostic", [Diagnos

[clang] [libclang/python] Do not rely on `ctypes`' `errcheck` (PR #105490)

2024-08-21 Thread Jannick Kremer via cfe-commits
@@ -2454,7 +2480,7 @@ def element_type(self): If accessed on a type that is not an array, complex, or vector type, an exception will be raised. """ -result = conf.lib.clang_getElementType(self) +result = Type.from_result(conf.lib.clang_ge

[clang] [clang][ASTImporter] set nonnull type for var specialized from lambda template (PR #105492)

2024-08-21 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 08decd20a968ddec459376ef64a4987ef74e15dc 2ac3aa1b85d727fd15d49fa1649147f2229576cb --e

[clang] [llvm] [InstCombine] Remove some of the complexity-based canonicalization (PR #91185)

2024-08-21 Thread Nikita Popov via cfe-commits
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/91185 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Balazs Benics via cfe-commits
https://github.com/steakhal created https://github.com/llvm/llvm-project/pull/105493 As discussed in https://discourse.llvm.org/t/rfc-make-istainted-and-complex-symbols-friends/79570/10 Some `isTainted()` queries can blow up the analysis times, and effectively halt the analysis under specific

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-static-analyzer-1 Author: Balazs Benics (steakhal) Changes As discussed in https://discourse.llvm.org/t/rfc-make-istainted-and-complex-symbols-friends/79570/10 Some `isTainted()` queries can blow up the analysis times, and effectively halt the an

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Balazs Benics via cfe-commits
https://github.com/steakhal edited https://github.com/llvm/llvm-project/pull/105493 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Balazs Benics via cfe-commits
steakhal wrote: Next release is scheduled for the 3rd of September. I want to backport this there too. https://github.com/llvm/llvm-project/pull/105493 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/list

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Gábor Horváth via cfe-commits
https://github.com/Xazax-hun approved this pull request. LGTM! https://github.com/llvm/llvm-project/pull/105493 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [RISCV][FMV] Support target_clones (PR #85786)

2024-08-21 Thread Piyou Chen via cfe-commits
https://github.com/BeMg updated https://github.com/llvm/llvm-project/pull/85786 >From 395ce72afbf9e4b12fcbfaf9cdbda8921c9ff72a Mon Sep 17 00:00:00 2001 From: Piyou Chen Date: Tue, 23 Jul 2024 19:59:06 -0700 Subject: [PATCH 1/4] [RISCV][FMV] Support target_clones --- .../clang/Basic/DiagnosticF

[clang] [clang] Add nuw attribute to GEPs (PR #105496)

2024-08-21 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-hlsl Author: Hari Limaye (hazzlim) Changes Add nuw attribute to inbounds GEPs where the expression used to form the GEP is an addition of unsigned indices. --- Patch is 576.17 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/l

[clang] [clang] Add nuw attribute to GEPs (PR #105496)

2024-08-21 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-codegen Author: Hari Limaye (hazzlim) Changes Add nuw attribute to inbounds GEPs where the expression used to form the GEP is an addition of unsigned indices. --- Patch is 576.17 KiB, truncated to 20.00 KiB below, full version: https://github.c

[clang] [clang] Add nuw attribute to GEPs (PR #105496)

2024-08-21 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Hari Limaye (hazzlim) Changes Add nuw attribute to inbounds GEPs where the expression used to form the GEP is an addition of unsigned indices. --- Patch is 576.17 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/

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

2024-08-21 Thread Budimir Aranđelović via cfe-commits
https://github.com/budimirarandjelovichtec updated https://github.com/llvm/llvm-project/pull/105479 From 8b5876fea6d3073332da972416c09602b81264af Mon Sep 17 00:00:00 2001 From: budimirarandjelovicsyrmia Date: Fri, 5 Apr 2024 15:20:37 +0200 Subject: [PATCH] [clang] Catch missing format attribute

[clang] [llvm] [Clang][AArch64] Add customisable immediate range checking to NEON (PR #100278)

2024-08-21 Thread via cfe-commits
@@ -369,12 +372,13 @@ class Intrinsic { public: Intrinsic(Record *R, StringRef Name, StringRef Proto, TypeSpec OutTS, TypeSpec InTS, ClassKind CK, ListInit *Body, NeonEmitter &Emitter, -StringRef ArchGuard, StringRef TargetGuard, bool IsUnavailable, b

[clang] [RISCV] Allow YAML file to control multilib selection (PR #98856)

2024-08-21 Thread Simon Cook via cfe-commits
simonpcook wrote: I think the fake flag hack may have a limitation in that since the `--XXX-fake-flag-...` appears in `--print-multi-lib` it may get imported into some build system and used as a flag for building particular libraries. For example with the above, newlib will not build since `er

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Donát Nagy via cfe-commits
https://github.com/NagyDonat edited https://github.com/llvm/llvm-project/pull/105493 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Donát Nagy via cfe-commits
@@ -459,7 +460,53 @@ unsigned radar11369570_hanging(const unsigned char *arr, int l) { longcmp(a, t, c); l -= 12; } - return 5/a; // expected-warning {{Division by a tainted value, possibly zero}} + return 5/a; // FIXME: Should be a "div by tainted" warning here.

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Donát Nagy via cfe-commits
https://github.com/NagyDonat approved this pull request. The change LGTM overall, I support this kind of mitigation. I added several cosmetic bikeshedding remarks as inline comments, but they are not important enough to block this PR. https://github.com/llvm/llvm-project/pull/105493 __

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Donát Nagy via cfe-commits
@@ -256,6 +257,12 @@ std::vector taint::getTaintedSymbolsImpl(ProgramStateRef State, if (!Sym) return TaintedSymbols; + // HACK:https://discourse.llvm.org/t/rfc-make-istainted-and-complex-symbols-friends/79570 + if (const auto &Opts = State->getAnalysisManager().getA

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Donát Nagy via cfe-commits
@@ -407,6 +407,11 @@ ANALYZER_OPTION( ANALYZER_OPTION(unsigned, MaxSymbolComplexity, "max-symbol-complexity", "The maximum complexity of symbolic constraint.", 35) +// HACK:https://discourse.llvm.org/t/rfc-make-istainted-and-complex-symbols-friends/79570 +// I

[clang] [RISCV] Allow YAML file to control multilib selection (PR #98856)

2024-08-21 Thread via cfe-commits
ArcaneNibble wrote: ... I had no idea that build systems might do that. Thanks, this probably explains why the feature was initially designed so that flags would be valid to pass as arguments https://github.com/llvm/llvm-project/pull/98856 ___ cfe-co

[clang] [Clang] Overflow Pattern Exclusions (PR #100272)

2024-08-21 Thread Nico Weber via cfe-commits
nico wrote: The test added here was tweaked a bit and then deleted in https://github.com/llvm/llvm-project/commit/07a8cbaf8dc16bebf6e875173d20299d9cc47cc5 What gives? Instead of deleting tests, we should revert the PR that adds them and then reland the PR with working tests. https://github.c

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Balazs Benics via cfe-commits
https://github.com/steakhal updated https://github.com/llvm/llvm-project/pull/105493 >From f9f481a55d174746205f33472e49410b7f57bc5a Mon Sep 17 00:00:00 2001 From: Balazs Benics Date: Wed, 21 Aug 2024 11:51:30 +0200 Subject: [PATCH 1/4] [analyzer] Limit `isTainted()` by skipping complicated sym

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Balazs Benics via cfe-commits
steakhal wrote: Fixed all remarks. Excellent feedback, thanks! https://github.com/llvm/llvm-project/pull/105493 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] ad435bc - [clang][CodeGen][SPIR-V][AMDGPU] Tweak AMDGCNSPIRV ABI to allow for the correct handling of aggregates passed to kernels / functions. (#102776)

2024-08-21 Thread via cfe-commits
Author: Alex Voicu Date: 2024-08-21T13:16:59+01:00 New Revision: ad435bcc14f42dc97286c717cd12446a0facb2ee URL: https://github.com/llvm/llvm-project/commit/ad435bcc14f42dc97286c717cd12446a0facb2ee DIFF: https://github.com/llvm/llvm-project/commit/ad435bcc14f42dc97286c717cd12446a0facb2ee.diff LO

[clang] [clang][CodeGen][SPIR-V][AMDGPU] Tweak AMDGCNSPIRV ABI to allow for the correct handling of aggregates passed to kernels / functions. (PR #102776)

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

[clang] [clang] Diagnose functions with too many parameters (PR #104833)

2024-08-21 Thread Vlad Serebrennikov via cfe-commits
https://github.com/Endilll updated https://github.com/llvm/llvm-project/pull/104833 >From 424818620766cedb2770e076ee359afeb0cc14ec Mon Sep 17 00:00:00 2001 From: Vlad Serebrennikov Date: Mon, 19 Aug 2024 19:26:46 +0300 Subject: [PATCH 1/7] Add in-loop check --- clang/include/clang/AST/Type.h

[clang] [clang] Diagnose functions with too many parameters (PR #104833)

2024-08-21 Thread Vlad Serebrennikov via cfe-commits
Endilll wrote: > @Endilll you forgot to update the tests - hence the ci failure Thank you! Should be good to go now https://github.com/llvm/llvm-project/pull/104833 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin

[clang] 8486589 - [analyzer] Limit `isTainted()` by skipping complicated symbols (#105493)

2024-08-21 Thread via cfe-commits
Author: Balazs Benics Date: 2024-08-21T14:24:56+02:00 New Revision: 848658955a9d2d42ea3e319d191e2dcd5d76c837 URL: https://github.com/llvm/llvm-project/commit/848658955a9d2d42ea3e319d191e2dcd5d76c837 DIFF: https://github.com/llvm/llvm-project/commit/848658955a9d2d42ea3e319d191e2dcd5d76c837.diff

[clang] [analyzer] Limit `isTainted()` by skipping complicated symbols (PR #105493)

2024-08-21 Thread Balazs Benics via cfe-commits
https://github.com/steakhal closed https://github.com/llvm/llvm-project/pull/105493 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Diagnose functions with too many parameters (PR #104833)

2024-08-21 Thread via cfe-commits
https://github.com/cor3ntin approved this pull request. https://github.com/llvm/llvm-project/pull/104833 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [AMDGPU] Convert AMDGPUResourceUsageAnalysis pass from Module to MF pass (PR #102913)

2024-08-21 Thread Janek van Oirschot via cfe-commits
@@ -68,82 +71,84 @@ void MCResourceInfo::assignMaxRegs() { assignMaxRegSym(MaxSGPRSym, MaxSGPR); } -void MCResourceInfo::finalize() { - assert(!finalized && "Cannot finalize ResourceInfo again."); - finalized = true; - assignMaxRegs(); +void MCResourceInfo::finalize(MCCon

[clang] [llvm] [BPF] introduce __attribute__((bpf_fastcall)) (PR #105417)

2024-08-21 Thread Nikita Popov via cfe-commits
nikic wrote: The last commit looks fine to me, I didn't look at the rest in detail assuming it was already reviewed. One thing I'm curious about is why this is implemented as a function attribute rather than as a calling convention. https://github.com/llvm/llvm-project/pull/105417 ___

[clang] [llvm] [BPF] introduce __attribute__((bpf_fastcall)) (PR #105417)

2024-08-21 Thread via cfe-commits
eddyz87 wrote: > The last commit looks fine to me, I didn't look at the rest in detail > assuming it was already reviewed. > > One thing I'm curious about is why this is implemented as a function > attribute rather than as a calling convention. Hi @nikic , Same question was raised on a previ

[libcxx] [libcxxabi] [libunwind] [runtimes] Always define cxx_shared, cxx_static & other targets (PR #80007)

2024-08-21 Thread Louis Dionne via cfe-commits
ldionne wrote: Ping @petrhosek @vitalybuka. I would like to merge this patch this week. CI seems to be happy, but your CI may be doing something different since IIRC this patch upset your bots the last time I tried landing it. Can you take it for a round to see if that's still the case? https

[clang] [Clang] [NFC] Introduce `DynamicRecursiveASTVisitor` (PR #105195)

2024-08-21 Thread Vlad Serebrennikov via cfe-commits
Endilll wrote: I wonder if we can reduce binary size by simply marking RAV as `visibility("hidden")` https://github.com/llvm/llvm-project/pull/105195 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listin

[clang] [clang] Add nuw attribute to GEPs (PR #105496)

2024-08-21 Thread Hari Limaye via cfe-commits
@@ -7,13 +7,6 @@ struct X { int a[2]; }; extern int bar(); -//. -// CHECK: @test.i23 = internal global i32 4, align 4 -// CHECK: @i = global i32 4, align 4 -// CHECK: @Arr = global [100 x i32] zeroinitializer, align 16 -// CHECK: @foo2.X = internal global ptr getelementptr (i8

[clang] [llvm] [AMDGPU] Convert AMDGPUResourceUsageAnalysis pass from Module to MF pass (PR #102913)

2024-08-21 Thread Janek van Oirschot via cfe-commits
@@ -3,6 +3,18 @@ declare i32 @llvm.amdgcn.workitem.id.x() +define <2 x i64> @f1() #0 { JanekvO wrote: There were some function order changes in some tests. This function is the same one as the function that is removed a couple of lines below. https://githu

[clang] [clang][ASTImporter] New fix for default template parameter values. (PR #101836)

2024-08-21 Thread Ding Fei via cfe-commits
=?utf-8?q?Balázs_Kéri?= Message-ID: In-Reply-To: @@ -5968,11 +5962,21 @@ ASTNodeImporter::VisitTemplateTypeParmDecl(TemplateTypeParmDecl *D) { } if (D->hasDefaultArgument()) { +// Default argument can be "inherited" when it has a reference to the +// previous d

[clang] Thread Safety Analysis: Differentiate between lock sets at real join points and expected/actual sets at function end (PR #105526)

2024-08-21 Thread Malek Ben Slimane via cfe-commits
https://github.com/malek203 created https://github.com/llvm/llvm-project/pull/105526 This fixes false positives related to returning a scoped lockable object. At the end of a function, we check managed locks instead of scoped locks. At real join points, we skip checking managed locks because w

[clang] 281d178 - [clang] Diagnose functions with too many parameters (#104833)

2024-08-21 Thread via cfe-commits
Author: Vlad Serebrennikov Date: 2024-08-21T17:38:24+04:00 New Revision: 281d17840c35a1d80303bb6170c253fe2411f95f URL: https://github.com/llvm/llvm-project/commit/281d17840c35a1d80303bb6170c253fe2411f95f DIFF: https://github.com/llvm/llvm-project/commit/281d17840c35a1d80303bb6170c253fe2411f95f.

[clang] [llvm] [Coverage][MC/DC] Show uncoverable and unreachable conditions (PR #94137)

2024-08-21 Thread via cfe-commits
Lambdaris wrote: I have no authority to request reviewers or commit this patch. So let's request someone, @chapuni @ornata @hanickadot https://github.com/llvm/llvm-project/pull/94137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lis

[clang] [clang][ASTImporter] set nonnull type for var specialized from lambda template (PR #105492)

2024-08-21 Thread Ding Fei via cfe-commits
https://github.com/danix800 updated https://github.com/llvm/llvm-project/pull/105492 >From 2ac3aa1b85d727fd15d49fa1649147f2229576cb Mon Sep 17 00:00:00 2001 From: dingfei Date: Wed, 21 Aug 2024 17:52:04 +0800 Subject: [PATCH 1/2] [clang][ASTImporter] set nonnull type for var specialized from l

[clang] [Sparc] Add flags to enable errata workaround pass for GR712RC and UT700 (PR #104742)

2024-08-21 Thread Daniel Cederman via cfe-commits
doac wrote: Our UT700 and GR712RC processors are in active production and we have customers that for various reasons wants to use clang instead of gcc. To do that they need to use these workarounds to avoid certain sensitive code sequences (the same workarounds are already available in gcc).

[clang] [clang][driver] Fix -print-target-triple OS version for apple targets (PR #104037)

2024-08-21 Thread Louis Dionne via cfe-commits
ldionne wrote: Thanks for the writeup, this is useful. This leaves me wondering if `arm64-apple-darwin22.4.0` might be the right target triple then, but the effective triple would be e.g. `arm64-apple-macos12.0.0`? https://github.com/llvm/llvm-project/pull/104037 __

[clang] [clang][ASTImporter] set nonnull type for var specialized from lambda template (PR #105492)

2024-08-21 Thread Ding Fei via cfe-commits
https://github.com/danix800 updated https://github.com/llvm/llvm-project/pull/105492 >From 2ac3aa1b85d727fd15d49fa1649147f2229576cb Mon Sep 17 00:00:00 2001 From: dingfei Date: Wed, 21 Aug 2024 17:52:04 +0800 Subject: [PATCH 1/3] [clang][ASTImporter] set nonnull type for var specialized from l

[clang] [llvm] [Coverage][MC/DC] Show uncoverable and unreachable conditions (PR #94137)

2024-08-21 Thread via cfe-commits
https://github.com/Lambdaris edited https://github.com/llvm/llvm-project/pull/94137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Implement P2747 constexpr placement new (PR #104586)

2024-08-21 Thread Aaron Ballman via cfe-commits
https://github.com/AaronBallman commented: Thank you for working on this! Given that it's an extension in C++20 mode, we're missing diagnostics and test coverage for issuing extension/precompat diagnostics. https://github.com/llvm/llvm-project/pull/104586 __

[clang] [Clang] Implement P2747 constexpr placement new (PR #104586)

2024-08-21 Thread Aaron Ballman via cfe-commits
@@ -245,7 +245,7 @@ namespace std { namespace PlacementNew { constexpr int foo() { // both-error {{never produces a constant expression}} char c[sizeof(int)]; -new (c) int{12}; // ref-note {{call to placement 'operator new'}} \ +new (c) int{12}; // ref-note {{this

[clang] [Clang] Implement P2747 constexpr placement new (PR #104586)

2024-08-21 Thread Aaron Ballman via cfe-commits
@@ -6691,7 +6691,7 @@ static bool HandleDestructionImpl(EvalInfo &Info, SourceRange CallRange, if (Size && Size > Value.getArrayInitializedElts()) expandArray(Value, Value.getArraySize() - 1); -for (; Size != 0; --Size) { +for (Size = Value.getArraySize(); S

[clang] [Clang] Implement P2747 constexpr placement new (PR #104586)

2024-08-21 Thread Aaron Ballman via cfe-commits
@@ -0,0 +1,78 @@ +// RUN: %clang_cc1 -std=c++2c -verify %s + + +namespace std { + using size_t = decltype(sizeof(0)); +} + +void *operator new(std::size_t, void *p) { return p; } +void* operator new[] (std::size_t, void* p) {return p;} + + +consteval int ok() { +int i; +n

[clang] [Clang] Implement P2747 constexpr placement new (PR #104586)

2024-08-21 Thread Aaron Ballman via cfe-commits
https://github.com/AaronBallman edited https://github.com/llvm/llvm-project/pull/104586 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

  1   2   3   4   >