chapuni wrote:
> > "Merge" facilities with std::max are removed as well for now.
>
> why?
They will be reintroduced later, "Any" in #121194 .
To be honest, I didn't imagine the way to preserve the old behavior.
For now, this change introduces "Merge" behavior partially.
https://github.com/llvm
@@ -1432,6 +1434,8 @@ CoverageData
CoverageMapping::getCoverageForFile(StringRef Filename) const {
getImpreciseRecordIndicesForFilename(Filename);
for (unsigned RecordIndex : RecordIndices) {
const FunctionRecord &Function = Functions[RecordIndex];
+if (Filtere
@@ -1432,6 +1434,8 @@ CoverageData
CoverageMapping::getCoverageForFile(StringRef Filename) const {
getImpreciseRecordIndicesForFilename(Filename);
for (unsigned RecordIndex : RecordIndices) {
const FunctionRecord &Function = Functions[RecordIndex];
+if (Filtere
ornata wrote:
>"Merge" facilities with std::max are removed as well for now.
why?
https://github.com/llvm/llvm-project/pull/121192
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/l
@@ -23,6 +23,16 @@ using namespace llvm;
namespace {
+template
+bool IsSummaryEmpty(const SummaryTy &Report, const CoverageViewOptions &Opts) {
ornata wrote:
case should be `isSummaryEmpty`
https://github.com/llvm/llvm-project/pull/121192
_
@@ -23,6 +23,16 @@ using namespace llvm;
namespace {
+template
+bool IsSummaryEmpty(const SummaryTy &Report, const CoverageViewOptions &Opts) {
ornata wrote:
function name case should be `isSummaryEmpty`
https://github.com/llvm/llvm-project/pull/121192
___
https://github.com/aemerson updated
https://github.com/llvm/llvm-project/pull/121171
>From 0be38ccf5c865b4fddc357b33c378c70a20532b9 Mon Sep 17 00:00:00 2001
From: Amara Emerson
Date: Thu, 26 Dec 2024 16:13:55 -0800
Subject: [PATCH 1/4] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20ch?=
=?
@@ -1400,14 +1375,12 @@ CodeGenFunction::EmitCXXForRangeStmt(const
CXXForRangeStmt &S,
if (ExitBlock != LoopExit.getBlock()) {
EmitBlock(ExitBlock);
+incrementProfileCounter(true, &S);
chapuni wrote:
Done. See also #120930.
https://github.com/llvm
@@ -864,8 +864,7 @@ void CodeGenFunction::EmitIfStmt(const IfStmt &S) {
// If the skipped block has no labels in it, just emit the executed block.
// This avoids emitting dead code and simplifies the CFG substantially.
if (S.isConstexpr() || !ContainsLabel(Skipped))
@@ -940,15 +939,35 @@ struct CounterCoverageMappingBuilder
std::pair getBranchCounterPair(const Stmt *S,
Counter ParentCnt) {
-Counter ExecCnt = getRegionCounter(S);
-return {ExecCnt, Builder.subtract(ParentCnt, Exec
@@ -1632,11 +1632,17 @@ class CodeGenFunction : public CodeGenTypeCache {
/// Increment the profiler's counter for the given statement by \p StepV.
/// If \p StepV is null, the default increment is 1.
void incrementProfileCounter(const Stmt *S, llvm::Value *StepV = nullpt
@@ -638,6 +638,10 @@ static unsigned getMaxCounterID(const
CounterMappingContext &Ctx,
unsigned MaxCounterID = 0;
for (const auto &Region : Record.MappingRegions) {
MaxCounterID = std::max(MaxCounterID, Ctx.getMaxCounterID(Region.Count));
+if (Region.Kind == Counte
@@ -112,6 +112,7 @@ class CodeGenPGO {
public:
std::pair getIsCounterPair(const Stmt *S) const;
chapuni wrote:
I couldn't name better.
```
return {I->second.Executed.hasValue(), I->second.Skipped.hasValue()};
```
It is defined in #112724.
https://github.com
@@ -1193,11 +1206,26 @@ std::pair
CodeGenPGO::getIsCounterPair(const Stmt *S) const {
}
void CodeGenPGO::emitCounterSetOrIncrement(CGBuilderTy &Builder, const Stmt *S,
+ bool UseSkipPath, bool UseBoth,
@@ -1193,11 +1206,26 @@ std::pair
CodeGenPGO::getIsCounterPair(const Stmt *S) const {
}
void CodeGenPGO::emitCounterSetOrIncrement(CGBuilderTy &Builder, const Stmt *S,
+ bool UseSkipPath, bool UseBoth,
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 ae36a7c2aa0626febb00b7bcf581a00ea0042d23
192d20b16e418b89a549e93406f782ac0a0fa093 --e
llvmbot wrote:
@llvm/pr-subscribers-backend-directx
Author: Justin Bogner (bogner)
Changes
This adds handling for raw and structured buffers when lowering resource access
via `llvm.dx.resource.getpointer`.
Fixes #121714
---
Patch is 25.43 KiB, truncated to 20.00 KiB below, full version
https://github.com/bogner created
https://github.com/llvm/llvm-project/pull/121725
This adds handling for raw and structured buffers when lowering resource access
via `llvm.dx.resource.getpointer`.
Fixes #121714
>From 192d20b16e418b89a549e93406f782ac0a0fa093 Mon Sep 17 00:00:00 2001
From: Jus
https://github.com/sharadhr updated
https://github.com/llvm/llvm-project/pull/121046
>From a9f921227abccda6c82137bed0b95041f584a147 Mon Sep 17 00:00:00 2001
From: Sharadh Rajaraman
Date: Tue, 24 Dec 2024 09:32:21 +
Subject: [PATCH 1/2] Accept /Fo and -Fo in `-fmodule-output` when running
u
19 matches
Mail list logo
