rjmccall added a comment. Thank you! I think all the invariants look right here; just some minor comments now, and then it should be ready to commit.
================ Comment at: clang/lib/CodeGen/CGExprComplex.cpp:403 + Builder.CreateStore(Val.second, ImagPtr, isVolatile); +} + ---------------- This looks to be dead code now, right? ================ Comment at: clang/lib/CodeGen/CGExprScalar.cpp:2860 + result = Builder.CreateFPTrunc(result, ConvertType(E->getType()), + "unpromotion"); + return result; ---------------- Please use your `EmitUnPromotedValue` function here (and in the similar functions below). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113107/new/ https://reviews.llvm.org/D113107 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits