madanial added a comment. In D154869#4628847 <https://reviews.llvm.org/D154869#4628847>, @efriedma wrote:
> Maybe split the changes to reformat the unittests into a separate patch? > > Otherwise, I'm happy with the current state of this patch, but probably > someone more active in flang should approve. I believe the changes are very trival to the unittests, would we be able to keep them as part of this patch and avoid moving it to github Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154869/new/ https://reviews.llvm.org/D154869 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits