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

[Peter Klausler via Phabricator via flang-commits]

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