https://github.com/jmorse closed
https://github.com/llvm/llvm-project/pull/111836
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jmorse wrote:
/me squints -- this was a minor improvement and I've lost all the context
behind it, abandoning as it's not worth revisiting IMO
https://github.com/llvm/llvm-project/pull/111836
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
@@ -51,7 +51,7 @@ class ConstraintSystem {
/// A map of variables (IR values) to their corresponding index in the
/// constraint system.
- DenseMap Value2Index;
+ SmallDenseMap Value2Index;
nikic wrote:
I'd add a using/typedef for this one, especially a
@@ -268,7 +268,7 @@ struct UniqueVirtualMethod {
/// subobject in which that virtual function occurs).
class OverridingMethods {
using ValuesT = SmallVector;
- using MapType = llvm::MapVector;
+ using MapType = llvm::SmallMapVector;
nikic wrote:
It looks l
jmorse wrote:
Undid the format-changes in this PR. For context: I've been building profiles
of all DenseMap allocations across a build of the CTMark suite to find
DenseMaps that a) are frequently used and b) usually have a small number of
elements, making them good candidates for initial stack
https://github.com/jmorse updated
https://github.com/llvm/llvm-project/pull/111836
>From d2b935e3a537e065b00f543a1792d1979ba413d9 Mon Sep 17 00:00:00 2001
From: Jeremy Morse
Date: Thu, 10 Oct 2024 14:17:34 +0100
Subject: [PATCH 1/3] [NFC] Replace more DenseMaps with SmallDenseMaps
These DenseM
https://github.com/kuhar commented:
Could you open a separate PR the code formatting changes? This will make it
easier to review.
https://github.com/llvm/llvm-project/pull/111836
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.ll
https://github.com/jmorse updated
https://github.com/llvm/llvm-project/pull/111836
>From d2b935e3a537e065b00f543a1792d1979ba413d9 Mon Sep 17 00:00:00 2001
From: Jeremy Morse
Date: Thu, 10 Oct 2024 14:17:34 +0100
Subject: [PATCH 1/2] [NFC] Replace more DenseMaps with SmallDenseMaps
These DenseM
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 774893dcd929c370bad714a70a7d670bb2d6f649
d2b935e3a537e065b00f543a1792d1979ba413d9 --e
llvmbot wrote:
@llvm/pr-subscribers-debuginfo
Author: Jeremy Morse (jmorse)
Changes
These DenseMaps all appear as some of the most frequent sources of
memory-allocations that could otherwise be accomodated with an initial stack
allocation. For simplicity, I've typedef'd one map-type to b
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-backend-x86
Author: Jeremy Morse (jmorse)
Changes
These DenseMaps all appear as some of the most frequent sources of
memory-allocations that could otherwise be accomodated with an initial stack
allocation. For simplicity, I'v
https://github.com/jmorse created
https://github.com/llvm/llvm-project/pull/111836
These DenseMaps all appear as some of the most frequent sources of
memory-allocations that could otherwise be accomodated with an initial stack
allocation. For simplicity, I've typedef'd one map-type to be Block
12 matches
Mail list logo
