[PATCH] D153281: [flang] add -flang-experimental-polymorphism flag to flang-new

2023-06-22 Thread David Truby via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGb81c5070d545: [flang] add -flang-experimental-polymorphism flag to flang-new (authored by DavidTruby). Changed prior to commit:

[PATCH] D153281: [flang] add -flang-experimental-polymorphism flag to flang-new

2023-06-20 Thread Valentin Clement via Phabricator via cfe-commits
clementval added a comment. Looks ok to me. Note that some of the polymorphic TODOs will only be removed when HLFIR is in place. Looks like you have a clang-format issue. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153281/new/

[PATCH] D153281: [flang] add -flang-experimental-polymorphism flag to flang-new

2023-06-20 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski accepted this revision. awarzynski added a comment. LGTM, thanks David! Could you also add a test in https://github.com/llvm/llvm-project/blob/main/flang/test/Driver/frontend-forwarding.f90? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D153281: [flang] add -flang-experimental-polymorphism flag to flang-new

2023-06-19 Thread Tom Eccles via Phabricator via cfe-commits
tblah accepted this revision. tblah added a comment. This revision is now accepted and ready to land. Looks good to me, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153281/new/ https://reviews.llvm.org/D153281

[PATCH] D153281: [flang] add -flang-experimental-polymorphism flag to flang-new

2023-06-19 Thread David Truby via Phabricator via cfe-commits
DavidTruby created this revision. DavidTruby added reviewers: tblah, vzakhari, awarzynski. Herald added a reviewer: sscalpone. Herald added a subscriber: sunshaoce. Herald added projects: Flang, All. DavidTruby requested review of this revision. Herald added subscribers: cfe-commits, jdoerfert,