================
@@ -95,6 +95,7 @@ CIRGenModule::CIRGenModule(mlir::MLIRContext &mlirContext,
   // TODO(CIR): Should be updated once TypeSizeInfoAttr is upstreamed
   const unsigned sizeTypeSize =
       astContext.getTypeSize(astContext.getSignedSizeType());
+  SizeSizeInBytes = astContext.toCharUnitsFromBits(sizeTypeSize).getQuantity();
----------------
andykaylor wrote:

The union is there to guarantee that both members have the same value, so we 
could set either one. The fact that I'm only using one in this PR is an 
artifact of the partial implementation, and the fact that the union is 
initialized with the one I'm not using is just a matter of reproducing the way 
it's done in the original code.

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

Reply via email to