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

[Gor Nishanov via llvm-commits]

GorNishanov added inline comments.

================
Comment at: lib/Transforms/Coroutines/CoroInstr.h:34
@@ +33,3 @@
+  enum { FrameArg, IndexArg };
+
+public:
----------------
How about now? I got rid of the macros and use plain strings in a call to declaresIntrinsic helper function.

To guard against mis-spelling, delaresIntrinsic in debug mode checks whether the name is a valid coroutine intrinsic name.


https://reviews.llvm.org/D22998