@@ -0,0 +1,60 @@
+#ifndef LLVM_CLANG_SUMMARY_SUMMARYCONTEXT_H
+#define LLVM_CLANG_SUMMARY_SUMMARYCONTEXT_H
+
+#include "clang/Summary/SummaryAttribute.h"
+#include "clang/Summary/SummaryConsumer.h"
+#include
+
+namespace clang {
+class FunctionSummary {
+ SmallVector ID;
---
https://github.com/Xazax-hun commented:
I did not have time to look at it properly, just some small drive by comments.
https://github.com/llvm/llvm-project/pull/144224
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-
@@ -0,0 +1,60 @@
+#ifndef LLVM_CLANG_SUMMARY_SUMMARYCONTEXT_H
+#define LLVM_CLANG_SUMMARY_SUMMARYCONTEXT_H
+
+#include "clang/Summary/SummaryAttribute.h"
+#include "clang/Summary/SummaryConsumer.h"
+#include
+
+namespace clang {
+class FunctionSummary {
+ SmallVector ID;
+ std:
llvmbot wrote:
@llvm/pr-subscribers-backend-risc-v
Author: Jim Lin (tclin914)
Changes
The spec can be found at:
https://github.com/andestech/andes-v5-isa/releases/tag/ast-v5_4_0-release.
This patch only supports assembler. The instructions are similar to `Zvfbfmin`
and the only differenc
nikic wrote:
This pass should have dedicated tests that don't mix in other coro passes.
https://github.com/llvm/llvm-project/pull/144319
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/c
https://github.com/tclin914 created
https://github.com/llvm/llvm-project/pull/144320
The spec can be found at:
https://github.com/andestech/andes-v5-isa/releases/tag/ast-v5_4_0-release.
This patch only supports assembler. The instructions are similar to `Zvfbfmin`
and the only difference with
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-mc
Author: Jim Lin (tclin914)
Changes
The spec can be found at:
https://github.com/andestech/andes-v5-isa/releases/tag/ast-v5_4_0-release.
This patch only supports assembler. The instructions are similar to `Zvfbfmin`
and the
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Jim Lin (tclin914)
Changes
The spec can be found at:
https://github.com/andestech/andes-v5-isa/releases/tag/ast-v5_4_0-release.
This patch only supports assembler. The instructions are similar to `Zvfbfmin`
and the only difference
https://github.com/smithp35 commented:
The linker workaround for 843419 is not particularly pleasant. I'm not sure how
much it triggers in real examples, but it will add to link time, particularly
for large programs.
For the amount of affected hardware out there (this erratum is 10 years old),
https://github.com/vvuksanovic updated
https://github.com/llvm/llvm-project/pull/142150
>From df2e394f94397bf5d371586703a70b8e9bec1aaa Mon Sep 17 00:00:00 2001
From: Vladimir Vuksanovic
Date: Fri, 30 May 2025 05:37:06 -0700
Subject: [PATCH] [clang-reorder-fields] Support designated initializers
mstorsjo wrote:
> I too had trouble understanding this change based on the description. Could
> you give a concrete example of how mingw and msvc disagree on where to put
> the attribute, and explain how this pr changes things?
>
> Taking a step back, how will this simplify libc++'s visibility
https://github.com/BaLiKfromUA edited
https://github.com/llvm/llvm-project/pull/144313
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/133699
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
philnik777 wrote:
> > I too had trouble understanding this change based on the description. Could
> > you give a concrete example of how mingw and msvc disagree on where to put
> > the attribute, and explain how this pr changes things?
```
template
class S {};
extern template class __declspe
davemgreen wrote:
> Personally I'd default to off, particularly after having clang default this
> to off for the last 10 years.
I didn't believe this changed the defaults (I may be mistaken, let me know if I
am!). It was already enabled on Android + Fuchsia, and I believe this patch
keeps tha
https://github.com/smithp35 edited
https://github.com/llvm/llvm-project/pull/143915
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
smithp35 wrote:
> > Personally I'd default to off, particularly after having clang default this
> > to off for the last 10 years.
>
> I didn't believe this changed the defaults (I may be mistaken, let me know if
> I am!). It was already enabled on Android + Fuchsia, and I believe this patch
>
philnik777 wrote:
Oh, also note that by default there will still be diagnostics about all
incorrect placements. You'll have to opt-in to silently ignoring incorrect
placements via `-Wno-dllexport-explicit-instantiation` (assuming I did things
correctly).
https://github.com/llvm/llvm-project/p
https://github.com/NewSigma created
https://github.com/llvm/llvm-project/pull/144319
As suggested by @ChuanqiXu9 in #14 , I propose LifetimeMovePass, which was
previously part of the CoroSplit pass, should now appear as a general pass.
Lifetime markers determine whether we place alloca on
https://github.com/someoneinjd edited
https://github.com/llvm/llvm-project/pull/142903
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
michael-jabbour-sonarsource wrote:
Hi @tristanlabelle and thanks for your response,
> 1. Code pervasively access files through canonicalized paths rather than
> using canonicalized paths only for identity comparisons. The two usages are
> often conflated and can't be teased apart easily.
I un
@@ -28,14 +28,15 @@ int main(int, char**) {
{
MoveOnly mo[] = {MoveOnly{3}};
// expected-error@array:* {{to_array requires copy constructible elements}}
-// expected-error-re@array:* (call to implicitly-deleted copy
constructor of 'MoveOnly')|(call to deleted
@@ -124,8 +126,8 @@ void test() {
#if _LIBCPP_HAS_EXCEPTIONS
// expected-error-re@*:* {{call to deleted constructor of{{.*
// expected-error-re@*:* {{call to deleted constructor of{{.*
- // expected-error-re@*:* {{call to deleted constructor of{{.*
- // expect
@@ -629,25 +629,46 @@ class ASTContext : public RefCountedBase {
void setRelocationInfoForCXXRecord(const CXXRecordDecl *,
CXXRecordDeclRelocationInfo);
- /// Examines a given type, and returns whether the T itself
+ /// Examines a give
@@ -629,25 +629,46 @@ class ASTContext : public RefCountedBase {
void setRelocationInfoForCXXRecord(const CXXRecordDecl *,
CXXRecordDeclRelocationInfo);
- /// Examines a given type, and returns whether the T itself
+ /// Examines a give
@@ -629,25 +629,46 @@ class ASTContext : public RefCountedBase {
void setRelocationInfoForCXXRecord(const CXXRecordDecl *,
CXXRecordDeclRelocationInfo);
- /// Examines a given type, and returns whether the T itself
+ /// Examines a give
bryanpkc wrote:
Thanks for the reviews @davemgreen and @smithp35.
https://github.com/llvm/llvm-project/pull/143915
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
egorshamshura wrote:
> Can you update the title/description to reflect the entire change (ie reuse
> the title/description of the previous PR)? Otherwise LGTM
>
> Thanks for fixing the libc++ tests
I have changed title and added description. How can I rerun job?
https://github.com/llvm/llvm-p
@@ -91,7 +91,9 @@ void fuchsia::Linker::ConstructJob(Compilation &C, const
JobAction &JA,
CmdArgs.push_back("--execute-only");
std::string CPU = getCPUName(D, Args, Triple);
-if (CPU.empty() || CPU == "generic" || CPU == "cortex-a53")
+if (Args.hasFlag(options
https://github.com/bryanpkc edited
https://github.com/llvm/llvm-project/pull/143915
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
401 - 430 of 430 matches
Mail list logo
