[PATCH] D22998: [coroutines] Part 4a: Coroutine Devirtualization: Lower coro.resume and coro.destroy.

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 31 12:00:05 PDT 2016


mehdi_amini added inline comments.

================
Comment at: lib/Transforms/Coroutines/CoroInternal.h:35
@@ +34,3 @@
+  Module& M;
+  LLVMContext& C;
+  FunctionType* const ResumeFnType;
----------------
`M` and `C` aren't great variable names. Also, is `C` used outside of the ctor? (The name does not help for grep...)


Repository:
  rL LLVM

https://reviews.llvm.org/D22998





More information about the llvm-commits mailing list