arphaman added inline comments.
================ Comment at: lib/CodeGen/CGBlocks.cpp:1414 + +} // end anonymous namespace + ---------------- vsk wrote: > I don't see the need for two GenericInfo types (although it's plausible it'll > make sense with your upcoming changes!). I had in mind a single 'enum > BlockCaptureOperationType' with 8 entries, and a 'struct > BlockCaptureOperation'. Agreed, fixed. Repository: rL LLVM https://reviews.llvm.org/D30345 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits