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

Theo de Magalhaes via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 4 11:00:06 PDT 2025


=?utf-8?q?Théo?= De Magalhaes <theo.de-magalhaes at epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalhaes at epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalhaes at epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalhaes at epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalhaes at epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalhaes at epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalhaes at epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalhaes at epita.fr>,Theo de
 Magalhaes <theodemagalhaes at icloud.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/134426 at github.com>


therealcoochieman wrote:

Fixes the issue in #130182 where the variable `RemainingBitsInField` was used uninitialized, leading to undefined behavior in certain cases.
Although I wasn't able to reproduce the undefined behavior using Clang 18.1.8, I validated the fix with Valgrind while running the windows-Wpadded.cpp test.

@mikaelholmen @asb, could you please confirm if this resolves the issue on your end?
Thanks for your patience, and I hope the pin isn't too disruptive.

https://github.com/llvm/llvm-project/pull/134426


More information about the cfe-commits mailing list