[PATCH] D77048: [Clang][CodeGen] Fixing mismatch between memory layout and const expressions for oversized bitfields

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 30 13:04:30 PDT 2020


efriedma added inline comments.


================
Comment at: clang/lib/CodeGen/CGExprConstant.cpp:613
+  
+  // Add padding bits in case of over-sized bit-field.
+  //   "The first sizeof(T)*8 bits are used to hold the value of the bit-field,
----------------
The existing code in ConstantAggregateBuilder::add should skip over padding.  I'm not sure what you're trying to accomplish by adding more code dealing with padding here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77048





More information about the cfe-commits mailing list