[clang] [CLANG][MS-STRUCT] bitfield padding warning presents padding to exact bit count (PR #136062)

2025-04-21 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= de Magalhaes Message-ID: In-Reply-To: theomagellan wrote: It doesn't affect non-ms_struct definitions! The new `PaddingInLastUnit` variable only gets a non-zero value when the following condition is met: `IsMsStruct && (LastBitfieldStorageUnitSize != StorageUnitSize ||

[clang] [CLANG][MS-STRUCT] bitfield padding warning presents padding to exact bit count (PR #136062)

2025-04-21 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= de Magalhaes Message-ID: In-Reply-To: https://github.com/theomagellan updated https://github.com/llvm/llvm-project/pull/136062 >From 842f0fbed0043ad0aa1679d8a30bc13d64eb25cf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Th=C3=A9o=20de=20Magalhaes?= Date: Thu, 17 Apr 2025 01:45:12

[clang] [CLANG][MS-STRUCT] bitfield padding warning presents padding to exact bit count (PR #136062)

2025-04-17 Thread Theo de Magalhaes via cfe-commits
theomagellan wrote: Hello @efriedma-quic, hope you don’t mind the ping. This patch addresses the padded bitfield warning, which was reporting the wrong number of padded bits when using `ms_struct` and two consecutive bit-fields under the Itanium ABI. I originally planned to merge `SemaCXX/win

[clang] [CLANG][MS-STRUCT] bitfield padding warning presents padding to exact bit count (PR #136062)

2025-04-17 Thread Theo de Magalhaes via cfe-commits
https://github.com/theomagellan ready_for_review https://github.com/llvm/llvm-project/pull/136062 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [CLANG][MS-STRUCT] bitfield padding warning presents padding to exact bit count (PR #136062)

2025-04-17 Thread Theo de Magalhaes via cfe-commits
https://github.com/theomagellan updated https://github.com/llvm/llvm-project/pull/136062 >From 842f0fbed0043ad0aa1679d8a30bc13d64eb25cf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Th=C3=A9o=20de=20Magalhaes?= Date: Thu, 17 Apr 2025 01:45:12 +0200 Subject: [PATCH] fix(ms_struct): bitfield padding w

[clang] [CLANG][MS-STRUCT] bitfield padding warning presents padding to exact bit count (PR #136062)

2025-04-16 Thread Theo de Magalhaes via cfe-commits
https://github.com/theomagellan converted_to_draft https://github.com/llvm/llvm-project/pull/136062 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [CLANG][MS-STRUCT] bitfield padding warning presents padding to exact bit count (PR #136062)

2025-04-16 Thread Theo de Magalhaes via cfe-commits
https://github.com/theomagellan created https://github.com/llvm/llvm-project/pull/136062 Aims to fix #131647. >From 842f0fbed0043ad0aa1679d8a30bc13d64eb25cf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Th=C3=A9o=20de=20Magalhaes?= Date: Thu, 17 Apr 2025 01:45:12 +0200 Subject: [PATCH] fix(ms_struc

[clang] [CLANG-CL] ignores Wpadded (PR #134426)

2025-04-08 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes ,Theo de Magalhaes , =?utf-8?q?Théo?= De Mag

[clang] [CLANG-CL] ignores Wpadded (PR #134426)

2025-04-08 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes ,Theo de Magalhaes , =?utf-8?q?Théo?= De Mag

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-04-05 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes Message-ID: In-Reply-To: https://github.c

[clang] [CLANG-CL] ignores Wpadded (PR #134426)

2025-04-04 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes ,Theo de Magalhaes , =?utf-8?q?Théo?= De Mag

[clang] [CLANG-CL] ignores Wpadded (PR #134426)

2025-04-04 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes ,Theo de Magalhaes , =?utf-8?q?Théo?= De Mag

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-04-04 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes Message-ID: In-Reply-To: @@ -0,0 +1,12 @@ +// RUN: %clang_cl -Wpadded -Wno-msvc-not-found -fsyntax-only -- %s 2>&1 | Fi

[clang] [CLANG-CL] ignores Wpadded (PR #134426)

2025-04-04 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes ,Theo de Magalhaes Message-ID: In-Reply-To:

[clang] [CLANG-CL] ignores Wpadded (PR #134426)

2025-04-04 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes ,Theo de Magalhaes Message-ID: In-Reply-To

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-04-03 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes Message-ID: In-Reply-To:

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-03-28 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes Message-ID: In-Reply-To: https://github.c

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-03-27 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes Message-ID: In-Reply-To: @@ -0,0 +1,12 @@ +// RUN: %clang_cl -Wpadded -Wno-msvc-not-foun

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-03-27 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes Message-ID: In-Reply-To: https://github.com/therealcoochieman updated https://github.com/llvm/llvm-proje

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-03-19 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes Message-ID: In-Reply-To: https://github.com/therealcoochieman updated https://github.com/llvm/llvm-project/pull/130182 >From 3a9bc908a7

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-03-18 Thread Theo de Magalhaes via cfe-commits
=?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes , =?utf-8?q?Théo?= De Magalhaes Message-ID: In-Reply-To: https://github.com/therealcoochieman updated https://github.com/llvm/llvm-project/pull/130182 >From 3a9bc908a791669c82f288ff745664411bf285ac M

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-03-07 Thread Theo de Magalhaes via cfe-commits
therealcoochieman wrote: > Please make sure to keep the formatting of the file as it was (you can format > the changes _you_ made to the file with `git clang-format`. > > > > Thanks! Hello, I misread the guidelines and thought I had to make a first clang-format commit before editing the fil

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-03-07 Thread Theo de Magalhaes via cfe-commits
https://github.com/therealcoochieman updated https://github.com/llvm/llvm-project/pull/130182 >From 3a9bc908a791669c82f288ff745664411bf285ac Mon Sep 17 00:00:00 2001 From: Theo de Magalhaes Date: Thu, 6 Mar 2025 22:26:37 +0100 Subject: [PATCH 1/3] chore(clang-format): formatted clang/lib/AST/R

[clang] [CLANG-CL] ignores wpadded (PR #130182)

2025-03-06 Thread Theo de Magalhaes via cfe-commits
https://github.com/therealcoochieman created https://github.com/llvm/llvm-project/pull/130182 Aims to fix #61702. I encountered an interesting behavior during my testing. It seems the Itanium Record Layout Builder can give out Wpadded-bitfield warnings with the wrong amount of padding when mi