[PATCH] D71899: [Coroutines][2/6] New pass manager: coro-split

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 26 08:41:25 PST 2019


jdoerfert added a comment.

As a generic comment: Changes like the addition of `const` or renaming of the pass to be `Legacy` could be done in NFC commits prior to this one without pre-commit review (IMHO). That makes this patch smaller. The same holds for splitting tests and functions, if there is no functional change.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71899/new/

https://reviews.llvm.org/D71899





More information about the llvm-commits mailing list