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

Reply via email to