[PATCH] D144870: [Clang][DebugInfo] Emit zero size bitfields in the debug info to delimit bitfields in different allocation units.

[Paul Robinson via Phabricator via cfe-commits]

probinson added inline comments.


================
Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:1558
+      EmitSeparator = FieldIt->isBitField();
+  }
+
----------------
I might not be following this correctly, but it feels like EmitSeparator will end up true if the last field is a bitfield, even if there are no zero-length bitfields in front of it. The test does not cover this case (to show that the no-zero-bitfields case is handled properly).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144870/new/

https://reviews.llvm.org/D144870