peixin added a comment. This needs rebase.
================ Comment at: flang/lib/Lower/Bridge.cpp:203 // them. for (Fortran::lower::pft::Program::Units &u : pft.getUnits()) { std::visit(Fortran::common::visitors{ ---------------- jpenix-quic wrote: > peixin wrote: > > Doing this can avoid add one variable to the bridge. > Done! (Although, I added the `isMainProgram()` check/update to the lambda > below as it is a member function of `FunctionLikeUnit` specifically--please > let me know if this looks ok!) Yes. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130513/new/ https://reviews.llvm.org/D130513 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits