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

2025-04-05 Thread Matheus Izvekov 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: mizvekov wrote:

[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 #130182)

2025-04-05 Thread Alex Bradbury 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-04-05 Thread Alex Bradbury 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-04-05 Thread Alex Bradbury 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: asb wrote: I'm

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

2025-04-05 Thread Mikael Holmén via cfe-commits
@@ -3004,6 +3010,15 @@ void MicrosoftRecordLayoutBuilder::layoutField(const FieldDecl *FD) { } else { FieldOffset = Size.alignTo(Info.Alignment); } + + uint64_t UnpaddedFielddOffsetInBits = + Context.toBits(DataSize) - RemainingBitsInField; mikae

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

2025-04-04 Thread Alex Bradbury via cfe-commits
=?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes Mess

[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 #130182)

2025-04-04 Thread Eli Friedman 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-04-04 Thread Alex Bradbury 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-04-03 Thread Eli Friedman 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: efriedma-quic wr

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

2025-04-03 Thread Alex Bradbury via cfe-commits
=?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes , =?utf-8?q?Th=C3=A9o?= De Magalhaes Mess

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

2025-04-03 Thread Alex Bradbury 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-04-03 Thread Alex Bradbury 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-04-03 Thread Mikael Holmén via cfe-commits
https://github.com/mikaelholmen edited https://github.com/llvm/llvm-project/pull/130182 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[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-04-03 Thread Mikael Holmén via cfe-commits
@@ -3004,6 +3010,15 @@ void MicrosoftRecordLayoutBuilder::layoutField(const FieldDecl *FD) { } else { FieldOffset = Size.alignTo(Info.Alignment); } + + uint64_t UnpaddedFielddOffsetInBits = + Context.toBits(DataSize) - RemainingBitsInField; mikae

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

2025-04-03 Thread Mikael Holmén via cfe-commits
mikaelholmen wrote: If I compile clang with clang (18.1.8) the new windows-Wpadded.cpp testcase fails for me like ``` error: 'expected-warning' diagnostics seen but not expected: File /repo/clang/test/SemaCXX/windows-Wpadded.cpp Line 20: padding struct 'Derived' with 2 bits to align 'c' 1 er

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

2025-04-02 Thread Mikael Holmén via cfe-commits
mikaelholmen wrote: > If I compile clang with clang (18.1.8) the new windows-Wpadded.cpp testcase > fails for me like > > ``` > error: 'expected-warning' diagnostics seen but not expected: > File /repo/clang/test/SemaCXX/windows-Wpadded.cpp Line 20: padding struct > 'Derived' with 2 bits to

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

2025-04-02 Thread 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: github-actions[b

[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-11 Thread via cfe-commits
github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

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

2025-03-07 Thread via cfe-commits
cor3ntin 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! https://github.com/llvm/llvm-project/pull/130182 ___ cfe-commits mailing list cfe-commit

[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 via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Theo de Magalhaes (therealcoochieman) Changes 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

[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