================
@@ -267,23 +267,60 @@ class AggValueSlot {
   Address addr;
   clang::Qualifiers quals;
 
+  /// This is set to true if some external code is responsible for setting up a
+  /// destructor for the slot.  Otherwise the code which constructs it should
+  /// push the appropriate cleanup.
+  [[maybe_unused]] bool destructedFlag : 1;
----------------
erichkeane wrote:

So using 'bool' on a bitfield is problematic and won't pack on some ABIs.  
These should all be `unsigned`.

Also, use `LLVM_PREFERRED_TYPE` to make it so diagnostics know they are bools.

https://github.com/llvm/llvm-project/pull/143932
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to