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

[Alex Bradbury via cfe-commits]

=?utf-8?q?Théo?= De Magalhaes <theo.de-magalh...@epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalh...@epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalh...@epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalh...@epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalh...@epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalh...@epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalh...@epita.fr>,
=?utf-8?q?Théo?= De Magalhaes <theo.de-magalh...@epita.fr>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/130...@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:

Ah, there __was__ an obvious reason. The GitHub PR reporting only kicks in if a 
builder built with this as the only commit that changed since the last build. 
The failing builders had more PRs than just this one in their changelist, so no 
comment on GitHub.

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