@@ -345,6 +345,10 @@ Bug Fixes in This Version
- Fixes an assertion failure on invalid code when trying to define member
functions in lambdas.
+- Clang now allows for member function templates of class templates declared
with a deduced return type
+ to be explicitly specia
https://github.com/sdkrystian updated
https://github.com/llvm/llvm-project/pull/86817
>From 468e3d9414a797ea73411a779343dee351e09e42 Mon Sep 17 00:00:00 2001
From: Krystian Stasiowski
Date: Wed, 27 Mar 2024 11:23:19 -0400
Subject: [PATCH 1/4] [Clang][Sema] Fix explicit specializations of member
https://github.com/python3kgae approved this pull request.
https://github.com/llvm/llvm-project/pull/87078
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bogner approved this pull request.
https://github.com/llvm/llvm-project/pull/87078
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jansvoboda11 wrote:
> If Clang is configured with 64 bit `SourceLocation`, we can't use 96 bits for
> serialization. We can at most use 64 bits for a slot. In that case, we can
> only assume the offset of source location **in its own module** (not the
> global offset!) is not large than 2^32.
https://github.com/bharadwajy approved this pull request.
https://github.com/llvm/llvm-project/pull/87078
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide updated
https://github.com/llvm/llvm-project/pull/84169
>From f0c1b2d114fc112726b61ec9274ba37cd3c2160b Mon Sep 17 00:00:00 2001
From: Sirraide
Date: Wed, 6 Mar 2024 14:50:02 +0100
Subject: [PATCH 1/2] [Clang] [Sema] No longer diagnose type definitions in
`offsetof`
Author: Sirraide
Date: 2024-03-29T17:42:09+01:00
New Revision: af34ac4508adcb5b54e62947d7caf7dd40227b16
URL:
https://github.com/llvm/llvm-project/commit/af34ac4508adcb5b54e62947d7caf7dd40227b16
DIFF:
https://github.com/llvm/llvm-project/commit/af34ac4508adcb5b54e62947d7caf7dd40227b16.diff
LOG:
https://github.com/Sirraide closed
https://github.com/llvm/llvm-project/pull/84169
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -345,6 +345,10 @@ Bug Fixes in This Version
- Fixes an assertion failure on invalid code when trying to define member
functions in lambdas.
+- Clang now allows for member function templates of class templates declared
with a deduced return type
+ to be explicitly specia
https://github.com/Sirraide approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/86817
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nickdesaulniers wrote:
> I think I am generally deeply confused about what should be provided by the
> compiler and what should be provided by the C Standard Library on any given
> platform.
+1
> Doing otherwise and trying to be "helpful" in Clang only creates confusion
> and forces C librar
Author: Cyndy Ishida
Date: 2024-03-29T10:00:51-07:00
New Revision: 60deb8b39afe9be90e30aa18d77ad129dacd4d55
URL:
https://github.com/llvm/llvm-project/commit/60deb8b39afe9be90e30aa18d77ad129dacd4d55
DIFF:
https://github.com/llvm/llvm-project/commit/60deb8b39afe9be90e30aa18d77ad129dacd4d55.diff
rjmccall wrote:
Hmm. The best solution there is probably to use a consistent representation
but introduce some sort of `GLValueThatTheStandardAbsurdlyPretendsIsAnRValue`
(definitely the only and best word for this, ship it) that we can use as the
value category. IIRC, something similar happe
https://github.com/urnathan created
https://github.com/llvm/llvm-project/pull/87090
This follows on from PR65742. As I suspected clipTailPadding's functionality
can be moved into accumulateBitfields; it has sufficient information now. It
turns out that clipTailPadding was undoing some of the
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Nathan Sidwell (urnathan)
Changes
This follows on from PR65742. As I suspected clipTailPadding's functionality
can be moved into accumulateBitfields; it has sufficient information now. It
turns out that clipTailPadding was undoing some o
@@ -418,39 +448,60 @@ static ReadySuspendResumeResult buildCoawaitCalls(Sema
&S, VarDecl *CoroPromise,
return Calls;
}
Expr *CoroHandle = CoroHandleRes.get();
- CallExpr *AwaitSuspend = cast_or_null(
- BuildSubExpr(ACT::ACT_Suspend, "await_suspend", CoroHandle));
michaelrj-google wrote:
Re: there being observable differences between a header in C vs C++ mode,
that's already a thing.
The definition for `isinf` in math.h is specified to be a macro in C, but
specified to be a function in C++. We're already doing `#ifdef __cplusplus` for
that (see
https:
https://github.com/nickdesaulniers approved this pull request.
https://github.com/llvm/llvm-project/pull/86742
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cachemeifyoucan approved this pull request.
https://github.com/llvm/llvm-project/pull/86980
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Farzon Lotfi
Date: 2024-03-29T13:19:28-04:00
New Revision: e74332a266e5f81411fb333bde56bf471f0a7ba6
URL:
https://github.com/llvm/llvm-project/commit/e74332a266e5f81411fb333bde56bf471f0a7ba6
DIFF:
https://github.com/llvm/llvm-project/commit/e74332a266e5f81411fb333bde56bf471f0a7ba6.diff
https://github.com/farzonl closed
https://github.com/llvm/llvm-project/pull/87078
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/11happy updated
https://github.com/llvm/llvm-project/pull/84481
>From 8fdf6306085ed4cf0f77b7e718e374e9f65fedf9 Mon Sep 17 00:00:00 2001
From: 11happy
Date: Fri, 8 Mar 2024 19:02:47 +0530
Subject: [PATCH 1/9] add clang-tidy check readability-math-missing-parentheses
Signed-of
@@ -0,0 +1,86 @@
+//===--- MathMissingParenthesesCheck.cpp - clang-tidy
-===//
+//
+// 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: Apa
@@ -0,0 +1,86 @@
+//===--- MathMissingParenthesesCheck.cpp - clang-tidy
-===//
+//
+// 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: Apa
@@ -0,0 +1,86 @@
+//===--- MathMissingParenthesesCheck.cpp - clang-tidy
-===//
+//
+// 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: Apa
philnik777 wrote:
> Re: there being observable differences between a header in C vs C++ mode,
> that's already a thing.
>
> The definition for `isinf` in math.h is specified to be a macro in C, but
> specified to be a function in C++. We're already doing `#ifdef __cplusplus`
> for that (see
Author: Cyndy Ishida
Date: 2024-03-29T10:34:35-07:00
New Revision: bdb60e6f0c8e89abf9bdf36411348db304ca65ba
URL:
https://github.com/llvm/llvm-project/commit/bdb60e6f0c8e89abf9bdf36411348db304ca65ba
DIFF:
https://github.com/llvm/llvm-project/commit/bdb60e6f0c8e89abf9bdf36411348db304ca65ba.diff
michaelrj-google wrote:
I'm fine with changing our definition to better coordinate with libc++, though
we should probably discuss that on someplace other than this patch.
https://github.com/llvm/llvm-project/pull/86748
___
cfe-commits mailing list
cfe
https://github.com/AtariDreams closed
https://github.com/llvm/llvm-project/pull/85481
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2331,6 +2332,11 @@ void CodeGenFunction::EmitVariablyModifiedType(QualType
type) {
type = cast(ty)->getPointeeType();
break;
+case Type::ArrayParameter:
efriedma-quic wrote:
Can we just use the existing "case" for ConstantArray?
https://
@@ -10926,6 +10965,10 @@ QualType ASTContext::mergeTypes(QualType LHS, QualType
RHS, bool OfBlockPointer,
assert(LHS != RHS &&
"Equivalent pipe types should have already been handled!");
return {};
+ case Type::ArrayParameter:
+assert(LHS != RHS &&
+
vgvassilev wrote:
Would it make sense to pass the eventual builtin includes (as string or
similar) to the interpreter constructor?
https://github.com/llvm/llvm-project/pull/87064
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.ll
https://github.com/tclin914 created
https://github.com/llvm/llvm-project/pull/87095
In M68kRegisterInfo.td, register SP is defined with name sp and alternate name
a7.
Fixes: https://github.com/llvm/llvm-project/issues/78620
>From dec6021133f67304bfc9942a1a4985cce6a15645 Mon Sep 17 00:00:00 20
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jim Lin (tclin914)
Changes
In M68kRegisterInfo.td, register SP is defined with name sp and alternate name
a7.
Fixes: https://github.com/llvm/llvm-project/issues/78620
---
Full diff: https://github.com/llvm/llvm-project/pull/87095.diff
https://github.com/vgvassilev approved this pull request.
Thank you!
https://github.com/llvm/llvm-project/pull/87066
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/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 d3bc9cc99b3d45e1fb8d65a57e308e899439fe26
dec6021133f67304bfc9942a1a4985cce6a15645 --
bogner wrote:
I have some misgivings about abusing the target intrinsic infrastructure like
this to create "target" intrinsics for a non-existent target. IMO this is
morally equivalent to just putting all of these intrinsics as generic llvm
intrinsics (`llvm.thread_id` or `llvm.all`), just wit
https://github.com/jansvoboda11 updated
https://github.com/llvm/llvm-project/pull/86358
>From 109e6039abe8bd3b598f1fdc2dbd7bca783ff4e6 Mon Sep 17 00:00:00 2001
From: Jan Svoboda
Date: Fri, 22 Mar 2024 15:54:14 -0700
Subject: [PATCH 1/5] [clang] Move state out of `PreprocessorOptions` (1/n)
An
Author: Jan Svoboda
Date: 2024-03-29T11:20:55-07:00
New Revision: 407a2f231a81862e20d80059870c48d818b61ec2
URL:
https://github.com/llvm/llvm-project/commit/407a2f231a81862e20d80059870c48d818b61ec2
DIFF:
https://github.com/llvm/llvm-project/commit/407a2f231a81862e20d80059870c48d818b61ec2.diff
L
https://github.com/jansvoboda11 closed
https://github.com/llvm/llvm-project/pull/86358
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ian-twilightcoder wrote:
@ldionne @AaronBallman is there anything I need to do before merging this?
https://github.com/llvm/llvm-project/pull/86748
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
https://github.com/jansvoboda11 created
https://github.com/llvm/llvm-project/pull/87099
An instance of `PreprocessorOptions` is part of `CompilerInvocation` which is
supposed to be a value type. The `FailedModules` member is problematic, since
it's essentially a shared state used by multiple `
ian-twilightcoder wrote:
I do wonder if we could have the broader builtin headers discussion independent
of this patch? Is everyone happy with this patch? We can keep talking about the
builtin headers in here independent of merging right?
https://github.com/llvm/llvm-project/pull/86748
___
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jan Svoboda (jansvoboda11)
Changes
An instance of `PreprocessorOptions` is part of `CompilerInvocation` which is
supposed to be a value type. The `FailedModules` member is problematic, since
it's essentially a shared state used by multipl
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 407a2f231a81862e20d80059870c48d818b61ec2
aa4f75ab41f5bc018c8c5a6352e8b9688249fb21 --
https://github.com/benlangmuir approved this pull request.
https://github.com/llvm/llvm-project/pull/87099
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jansvoboda11 updated
https://github.com/llvm/llvm-project/pull/87099
>From aa4f75ab41f5bc018c8c5a6352e8b9688249fb21 Mon Sep 17 00:00:00 2001
From: Jan Svoboda
Date: Fri, 22 Mar 2024 15:08:07 -0700
Subject: [PATCH 1/2] [clang] Move state out of `PreprocessorOptions` (2/n)
An
Author: Jan Svoboda
Date: 2024-03-29T12:07:47-07:00
New Revision: ee3a302abaa091e550a80f79694e963d1b5d0b7b
URL:
https://github.com/llvm/llvm-project/commit/ee3a302abaa091e550a80f79694e963d1b5d0b7b
DIFF:
https://github.com/llvm/llvm-project/commit/ee3a302abaa091e550a80f79694e963d1b5d0b7b.diff
L
https://github.com/jansvoboda11 closed
https://github.com/llvm/llvm-project/pull/87099
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,618 @@
+; RUN: llc -mtriple=amdgcn -mcpu=gfx900 -mattr=+precise-memory < %s |
FileCheck %s -check-prefixes=GFX9
+; RUN: llc -mtriple=amdgcn -mcpu=gfx90a -mattr=+precise-memory < %s |
FileCheck %s -check-prefixes=GFX90A
+; RUN: llc -mtriple=amdgcn -mcpu=gfx1010 -mattr=
https://github.com/AtariDreams reopened
https://github.com/llvm/llvm-project/pull/86967
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AtariDreams updated
https://github.com/llvm/llvm-project/pull/86967
>From 5d8873d4c296a839674f4ad1a51c779df9c68fab Mon Sep 17 00:00:00 2001
From: Rose
Date: Thu, 28 Mar 2024 12:12:57 -0400
Subject: [PATCH] [CGBuiltin] Use freeze instruction to create an undef value
instead o
https://github.com/AtariDreams closed
https://github.com/llvm/llvm-project/pull/85487
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
urnathan wrote:
Sigh, clipTailPadding is still needed, because of no_unique_address and empty
base placement. Will update.
https://github.com/llvm/llvm-project/pull/87090
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/
https://github.com/AtariDreams converted_to_draft
https://github.com/llvm/llvm-project/pull/86967
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sopyb wrote:
Hey there, I've hit a bit of a wall. I'm trying to figure out if a location is
inside a comment. The check generates faulty diagnostics, as it is now, if
there are comments including '(', '{', '}', and ','. If I could check if the
match is inside a comment this issue could be fixe
https://github.com/llvm-beanz updated
https://github.com/llvm/llvm-project/pull/79382
>From bd74d7db681cd07fda56f26e79e047c6d1a41f6b Mon Sep 17 00:00:00 2001
From: Chris Bieneman
Date: Thu, 29 Feb 2024 15:37:50 -0600
Subject: [PATCH 1/6] [HLSL] Pass arrays by value
HLSL constant sized array fu
https://github.com/topperc closed
https://github.com/llvm/llvm-project/pull/86967
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
topperc wrote:
I believe this has been tried in the past and reverted. Have we resolved the
issues from the past?
https://github.com/llvm/llvm-project/pull/86967
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/ma
https://github.com/topperc reopened
https://github.com/llvm/llvm-project/pull/86967
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/llvm-beanz updated
https://github.com/llvm/llvm-project/pull/79382
>From bd74d7db681cd07fda56f26e79e047c6d1a41f6b Mon Sep 17 00:00:00 2001
From: Chris Bieneman
Date: Thu, 29 Feb 2024 15:37:50 -0600
Subject: [PATCH 1/7] [HLSL] Pass arrays by value
HLSL constant sized array fu
overmighty wrote:
Could someone merge this for me if it looks good to everyone?
https://github.com/llvm/llvm-project/pull/86742
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2326,6 +2326,20 @@ bool
SIInsertWaitcnts::insertWaitcntInBlock(MachineFunction &MF,
}
#endif
+if (ST->isPreciseMemoryEnabled() && Inst.mayLoadOrStore()) {
+ AMDGPU::Waitcnt Wait;
+ if (ST->hasExtendedWaitCounts())
+Wait = AMDGPU::Waitcnt(0, 0, 0,
https://github.com/jhuber6 created
https://github.com/llvm/llvm-project/pull/87108
Summary:
We want to pass these GPU libraries by default if a certain offloading
toolchain is loaded for OpenMP. Previously I parsed this from the
arguments because it's only available in the compilation. This does
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Joseph Huber (jhuber6)
Changes
Summary:
We want to pass these GPU libraries by default if a certain offloading
toolchain is loaded for OpenMP. Previously I parsed this from the
arguments because it's only available in the compilation. This
dhruvachak wrote:
Some OMPT variables with the same name used to be present in both libomptarget
and the common plugin interface. These should probably be re-worked in the new
scheme of static linking? e.g.
llvm::omp::target::ompt::Initialized
https://github.com/llvm/llvm-project/pull/87009
_
dmpots wrote:
I did not see any discussion of local root signatures here. It is probably good
to consider those as well when thinking about this design. There is alternate
syntax for specifying both local root signatues and global root signatures
starting in SM 6.3
https://learn.microsoft.com
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/87108
>From 4415c4d4b9c72e963d4c483440598933d59e19cc Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Fri, 29 Mar 2024 15:25:00 -0500
Subject: [PATCH] [OpenMP] Use loaded offloading toolchains to add libraries
Summar
jhuber6 wrote:
> Some OMPT variables with the same name used to be present in both
> libomptarget and the common plugin interface. These should probably be
> re-worked in the new scheme of static linking? e.g.
> llvm::omp::target::ompt::Initialized
Unsure, there was an issue where each plugin
https://github.com/farzonl closed
https://github.com/llvm/llvm-project/pull/87034
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Cyndy Ishida
Date: 2024-03-29T13:44:22-07:00
New Revision: 12fdf04ff8962c9e902669a1b600b27a960f0c11
URL:
https://github.com/llvm/llvm-project/commit/12fdf04ff8962c9e902669a1b600b27a960f0c11
DIFF:
https://github.com/llvm/llvm-project/commit/12fdf04ff8962c9e902669a1b600b27a960f0c11.diff
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/86980
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
stevecor wrote:
Ping
https://github.com/llvm/llvm-project/pull/77021
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/efriedma-quic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/79382
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: OverMighty
Date: 2024-03-29T14:01:19-07:00
New Revision: cf73f136c5e22e7e4090fc762456ace47752a898
URL:
https://github.com/llvm/llvm-project/commit/cf73f136c5e22e7e4090fc762456ace47752a898
DIFF:
https://github.com/llvm/llvm-project/commit/cf73f136c5e22e7e4090fc762456ace47752a898.diff
LO
https://github.com/efriedma-quic closed
https://github.com/llvm/llvm-project/pull/86742
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2326,6 +2326,20 @@ bool
SIInsertWaitcnts::insertWaitcntInBlock(MachineFunction &MF,
}
#endif
+if (ST->isPreciseMemoryEnabled() && Inst.mayLoadOrStore()) {
+ AMDGPU::Waitcnt Wait;
+ if (ST->hasExtendedWaitCounts())
+Wait = AMDGPU::Waitcnt(0, 0, 0,
https://github.com/vvereschaka created
https://github.com/llvm/llvm-project/pull/87113
Review the actual component parameters and update the cache file accordingly.
Also fixed the C++ test builds for the compiler-rt component.
>From bf660e351d605db3062d782c6f526ac062548803 Mon Sep 17 00:00:00
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Vladimir Vereschaka (vvereschaka)
Changes
Review the actual component parameters and update the cache file accordingly.
Also fixed the C++ test builds for the compiler-rt component.
---
Full diff: https://github.com/llvm/llvm-project/pull
@@ -3043,10 +3043,6 @@ struct AMDGPUPluginTy final : public GenericPluginTy {
// HSA functions from now on, e.g., hsa_shut_down.
Initialized = true;
-#ifdef OMPT_SUPPORT
-ompt::connectLibrary();
dhruvachak wrote:
There is a connectLibrary definiti
https://github.com/coopp approved this pull request.
WOW.. This was pretty big. I went through it twice and didn't see anything
that jumped out. Looks good to me.
https://github.com/llvm/llvm-project/pull/79382
___
cfe-commits mailing list
cfe-commi
@@ -0,0 +1,145 @@
+// REQUIRES: lld-available
+
+// RUN: rm -rf %t && split-file %s %t && cd %t
+//
+// RUN: %clangxx_pgogen -fuse-ld=lld -O2 -g -fprofile-generate=. -mllvm
-enable-vtable-value-profiling test.cpp -o test
+// RUN: env LLVM_PROFILE_FILE=test.profraw ./test
@@ -431,23 +441,34 @@ class InstrProfSymtab {
using AddrHashMap = std::vector>;
private:
+ using AddrIntervalMap =
+ IntervalMap>;
StringRef Data;
uint64_t Address = 0;
- // Unique name strings.
+ // Unique name strings. Used to ensure entries in MD5NameMap (a
@@ -676,24 +722,66 @@ TEST_P(InstrProfReaderWriterTest, icall_data_read_write) {
Expected R = Reader->getInstrProfRecord("caller", 0x1234);
ASSERT_THAT_ERROR(R.takeError(), Succeeded());
+
+ // Test the number of instrumented indirect call sites and the number of
+ // pr
@@ -0,0 +1,145 @@
+// REQUIRES: lld-available
+
+// RUN: rm -rf %t && split-file %s %t && cd %t
+//
+// RUN: %clangxx_pgogen -fuse-ld=lld -O2 -g -fprofile-generate=. -mllvm
-enable-vtable-value-profiling test.cpp -o test
+// RUN: env LLVM_PROFILE_FILE=test.profraw ./test
+
+// Sh
@@ -676,24 +722,66 @@ TEST_P(InstrProfReaderWriterTest, icall_data_read_write) {
Expected R = Reader->getInstrProfRecord("caller", 0x1234);
ASSERT_THAT_ERROR(R.takeError(), Succeeded());
+
+ // Test the number of instrumented indirect call sites and the number of
+ // pr
@@ -0,0 +1,145 @@
+// REQUIRES: lld-available
+
+// RUN: rm -rf %t && split-file %s %t && cd %t
+//
+// RUN: %clangxx_pgogen -fuse-ld=lld -O2 -g -fprofile-generate=. -mllvm
-enable-vtable-value-profiling test.cpp -o test
+// RUN: env LLVM_PROFILE_FILE=test.profraw ./test
+
+// Sh
@@ -0,0 +1,145 @@
+// REQUIRES: lld-available
+
+// RUN: rm -rf %t && split-file %s %t && cd %t
minglotus-6 wrote:
done.
https://github.com/llvm/llvm-project/pull/66825
___
cfe-commits mailing list
cfe-commits@lists.ll
@@ -1362,8 +1372,8 @@ remapSamples(const sampleprof::FunctionSamples &Samples,
BodySample.second.getSamples());
for (const auto &Target : BodySample.second.getCallTargets()) {
Result.addCalledTargetSamples(BodySample.first.LineOffset,
-
Author: Vladimir Vereschaka
Date: 2024-03-29T15:31:31-07:00
New Revision: b4693e0d7da8f7f1f1365ab97ecc0e33ad459943
URL:
https://github.com/llvm/llvm-project/commit/b4693e0d7da8f7f1f1365ab97ecc0e33ad459943
DIFF:
https://github.com/llvm/llvm-project/commit/b4693e0d7da8f7f1f1365ab97ecc0e33ad459943
https://github.com/vvereschaka closed
https://github.com/llvm/llvm-project/pull/87113
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
justdan96 wrote:
No problem, I have wasi-sdk all working in a Docker container now with your
latest version of wasm-component-ld so I should be thanking you!
https://github.com/llvm/llvm-project/pull/84569
___
cfe-commits mailing list
cfe-commits@list
@@ -3043,10 +3043,6 @@ struct AMDGPUPluginTy final : public GenericPluginTy {
// HSA functions from now on, e.g., hsa_shut_down.
Initialized = true;
-#ifdef OMPT_SUPPORT
-ompt::connectLibrary();
jhuber6 wrote:
Ah, well that can be a follow-up patc
a1batross wrote:
I have built clang-format with these changes. On current commit, it sadly
doesn't remove the space in these patterns:
```c
memset( g_used, 0, sizeof( g_used ) );
if( FBitSet( ent->curstate.effects, EF_INVLIGHT ) )
```
After reverting last commit, it got somewhat closer:
```c
me
https://github.com/MaskRay created
https://github.com/llvm/llvm-project/pull/87130
https://reviews.llvm.org/D54188 marked "alias" targets as used in C to
fix -Wunused false positives. This patch extends the approach to
handle mangled names to support global scope names in C++. We fix false
posit
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Fangrui Song (MaskRay)
Changes
https://reviews.llvm.org/D54188 marked "alias" targets as used in C to
fix -Wunused false positives. This patch extends the approach to
handle mangled names to support global scope names in C++. We fix false
p
https://github.com/marcauberer created
https://github.com/llvm/llvm-project/pull/87131
Resolves #87109
>From 68d12152e79f3507accb3887f275a98a7976ad5e Mon Sep 17 00:00:00 2001
From: Marc Auberer
Date: Sat, 30 Mar 2024 01:52:52 +0100
Subject: [PATCH] [Clang] Move hlsl_wave_get_lane_index to Emit
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Marc Auberer (marcauberer)
Changes
Resolves #87109
---
Full diff: https://github.com/llvm/llvm-project/pull/87131.diff
1 Files Affected:
- (modified) clang/lib/CodeGen/CGBuiltin.cpp (+8-10)
``diff
diff --git a/clang/lib/CodeG
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Marc Auberer (marcauberer)
Changes
Resolves #87109
---
Full diff: https://github.com/llvm/llvm-project/pull/87131.diff
1 Files Affected:
- (modified) clang/lib/CodeGen/CGBuiltin.cpp (+8-10)
``diff
diff --git a/clang/l
101 - 200 of 218 matches
Mail list logo
