https://github.com/urnathan closed
https://github.com/llvm/llvm-project/pull/89055
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/urnathan updated
https://github.com/llvm/llvm-project/pull/89055
>From 9ab483f3451bfcaa7968c5f1cf7115144522f58a Mon Sep 17 00:00:00 2001
From: Nathan Sidwell
Date: Mon, 1 Apr 2024 16:15:12 -0400
Subject: [PATCH 1/3] [clang] Remove class layout scissor
---
https://github.com/urnathan updated
https://github.com/llvm/llvm-project/pull/89055
>From db5e6456f26ea9b859d3ff24161d7494d58bb7e1 Mon Sep 17 00:00:00 2001
From: Nathan Sidwell
Date: Mon, 1 Apr 2024 16:15:12 -0400
Subject: [PATCH 1/3] [clang] Remove class layout scissor
---
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/89055
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
Might be useful to mention #87090 in the commit message (first comment).
https://github.com/llvm/llvm-project/pull/89055
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -950,18 +948,20 @@ void CGRecordLowering::calculateZeroInit() {
}
// Verify accumulateBitfields computed the correct storage representations.
-void CGRecordLowering::checkBitfieldClipping() const {
+void CGRecordLowering::checkBitfieldClipping(
+bool
urnathan wrote:
ping?
https://github.com/llvm/llvm-project/pull/89055
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Nathan Sidwell (urnathan)
Changes
Now that `accumulateBitfields` does the correct clipping, we don't need the
scissor any more -- `checkBitfieldClipping` can compute its location directly.
---
Full diff:
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Nathan Sidwell (urnathan)
Changes
Now that `accumulateBitfields` does the correct clipping, we don't need the
scissor any more -- `checkBitfieldClipping` can compute its location directly.
---
Full diff:
https://github.com/urnathan ready_for_review
https://github.com/llvm/llvm-project/pull/89055
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/urnathan updated
https://github.com/llvm/llvm-project/pull/89055
>From db5e6456f26ea9b859d3ff24161d7494d58bb7e1 Mon Sep 17 00:00:00 2001
From: Nathan Sidwell
Date: Mon, 1 Apr 2024 16:15:12 -0400
Subject: [PATCH 1/2] [clang] Remove class layout scissor
---
https://github.com/urnathan created
https://github.com/llvm/llvm-project/pull/89055
Now that `accumulateBitfields` does the correct clipping, we don't need the
scissor any more -- `checkBitfieldClipping` can compute its location directly.
>From db5e6456f26ea9b859d3ff24161d7494d58bb7e1 Mon Sep
12 matches
Mail list logo
