koplas wrote:
Using `-fgen-reduced-bmi` currently fails for me with a case that looks like
this:
```C++
// a.hpp
template
void ignore(T const &) noexcept {}
inline void resultCheck(char const *message) {
ignore(message);
}
// b.cppm
module;
#include "a.hpp"
expor
koplas wrote:
Do I miss something? The performance and file size is similar with and without
`-fgen-reduced-bmi`.
To reproduce clone https://github.com/koplas/clang-modules-test and run
`build.sh` and `build_thin_bmi.sh`.
https://github.com/llvm/llvm-project/pull/85050
https://github.com/koplas created
https://github.com/llvm/llvm-project/pull/132802
The File ID is incorrectly calculated, resulting in an out-of-bounds access.
The test code is more complex because the File fetching only happens in
specific scenarios.
>From cad802f9c91f12b8db26bc63eef13097474
https://github.com/koplas updated
https://github.com/llvm/llvm-project/pull/132802
>From 2d143ca15476df04063e9e7e2c5fd1938c4f705b Mon Sep 17 00:00:00 2001
From: koplas
Date: Mon, 24 Mar 2025 19:20:52 +0100
Subject: [PATCH 1/2] [PATCH] [clang][modules] Fix serialization and
de-serialization of
@@ -2601,10 +2601,11 @@ class CXXConstructorDecl final
void anchor() override;
size_t numTrailingObjects(OverloadToken) const {
-return CXXConstructorDeclBits.IsInheritingConstructor;
+return getCanonicalDecl()->CXXConstructorDeclBits.IsInheritingConstructor;
}
https://github.com/koplas edited
https://github.com/llvm/llvm-project/pull/133077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/koplas edited
https://github.com/llvm/llvm-project/pull/133077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/koplas created
https://github.com/llvm/llvm-project/pull/133077
When getting the `ExplicitSpecifier` of the `CXXConstructorDecl`, one of the
canonical declaration is returned. When writing the declaration record with
`ExplicitSpecifier`, the `AllocKind` of the canonical decl
https://github.com/koplas updated
https://github.com/llvm/llvm-project/pull/133077
>From d2da89cb6d2438b31ad25a05ec87d5f039e79064 Mon Sep 17 00:00:00 2001
From: koplas
Date: Wed, 26 Mar 2025 13:49:44 +0100
Subject: [PATCH] [PATCH] [clang][frontend] Fix AllocKind retrieving for
`CXXConstructorD
https://github.com/koplas updated
https://github.com/llvm/llvm-project/pull/133077
>From 86033b4b698ad11cf358b929e950f0b01e50390e Mon Sep 17 00:00:00 2001
From: koplas
Date: Wed, 26 Mar 2025 13:49:44 +0100
Subject: [PATCH] [PATCH] [clang][frontend] Fix AllocKind retrieval for
CXXConstructorDec
https://github.com/koplas edited
https://github.com/llvm/llvm-project/pull/133077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/koplas edited
https://github.com/llvm/llvm-project/pull/133077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/koplas updated
https://github.com/llvm/llvm-project/pull/133077
>From 686a9a772400316e5597409c7bd14f4d73e353bf Mon Sep 17 00:00:00 2001
From: koplas
Date: Wed, 26 Mar 2025 13:49:44 +0100
Subject: [PATCH] [PATCH] [clang][frontend] Fix AllocKind retrieving for
`CXXConstructorD
https://github.com/koplas ready_for_review
https://github.com/llvm/llvm-project/pull/133077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/koplas updated
https://github.com/llvm/llvm-project/pull/133077
>From 81df4736822848b35959aa067fc9d056ed868946 Mon Sep 17 00:00:00 2001
From: koplas
Date: Thu, 27 Mar 2025 08:00:54 +0100
Subject: [PATCH 1/2] [PATCH] [clang][frontend] Fix serialization for
CXXConstructorDecl
koplas wrote:
> LGTM, Thanks! Will you need me to merge that for you?
Yes, I don't have any permissions to merge this.
https://github.com/llvm/llvm-project/pull/133077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-
https://github.com/koplas updated
https://github.com/llvm/llvm-project/pull/133077
>From 81df4736822848b35959aa067fc9d056ed868946 Mon Sep 17 00:00:00 2001
From: koplas
Date: Thu, 27 Mar 2025 08:00:54 +0100
Subject: [PATCH] [PATCH] [clang][frontend] Fix serialization for
CXXConstructorDecl (ref
https://github.com/koplas edited
https://github.com/llvm/llvm-project/pull/133077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
koplas wrote:
@shafik, I updated the commit message based on the feedback of @cor3ntin. I
have now updated the title and description to reflect the new commit message,
to avoid further confusion.
https://github.com/llvm/llvm-project/pull/133077
___
c
https://github.com/koplas edited
https://github.com/llvm/llvm-project/pull/133077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/koplas edited
https://github.com/llvm/llvm-project/pull/133077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/koplas edited
https://github.com/llvm/llvm-project/pull/133077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
22 matches
Mail list logo
