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

Peter Klausler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 29 08:03:44 PDT 2020


klausler abandoned this revision.
klausler added a comment.

In D82695#2119300 <https://reviews.llvm.org/D82695#2119300>, @richard.barton.arm wrote:

> 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


I'm happy to not push this code if you don't want it.


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