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

[Alex Bradbury via cfe-commits]

=?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>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/130182 at github.com>


================
@@ -3004,6 +3010,15 @@ void MicrosoftRecordLayoutBuilder::layoutField(const FieldDecl *FD) {
   } else {
     FieldOffset = Size.alignTo(Info.Alignment);
   }
+
+  uint64_t UnpaddedFielddOffsetInBits =
+      Context.toBits(DataSize) - RemainingBitsInField;
----------------
asb wrote:

Thank you both! In line with our patch reversion policy https://llvm.org/docs/DeveloperPolicy.html#patch-reversion-policy I've gone ahead and reverted so that the issue can be fixed without undue time pressure, and so bots can go green again in the meanwhile.

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