================ @@ -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