[PATCH] D100651: [AIX] Support of Big archive (read)
    Jinsong Ji via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Oct 12 06:48:15 PDT 2021
    
    
  
jsji added a comment.
> In my opinion, this PR is complex enough. It might be accepted with as few changes as possible. So, without freelist, undocumented features... In a second time, you can create a new PR, to extend support of Big Archive, with new tests.
As @EGuesnet mentioned above, I think he won't have time to address the comments in time.
However, we do depend on this patch for AIX in near future, 
so I would suggest that we *accept* this patch *as it is* for now, and commit it as @EGuesnet suggested.
And @DiggerLin will post follow up patches immediately.
Is that OK for reviewers? @jhenderson @rupprecht @Esme ?
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100651/new/
https://reviews.llvm.org/D100651
    
    
More information about the llvm-commits
mailing list