[PATCH] D82695: [flang] Roll up work on external I/O runtime library

Richard Barton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 29 01:34:37 PDT 2020


richard.barton.arm requested changes to this revision.
richard.barton.arm added a comment.
This revision now requires changes to proceed.

Please can this patch be split up into separate patches for each of the separate types changes? Roll-up patches are not good practice and against the developer policy <https://llvm.org/docs/DeveloperPolicy.html#incremental-development>. 
@PeteSteinfeld The patches should also be code reviewed for LLVM master. Prior review on some other project is not relevant and should not be given as justification for approving patches to LLVM master.
The clang-tidy linting errors also need fixing up.

Thanks
Rich


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82695





More information about the llvm-commits mailing list