[PATCH] D143478: [RFC][Flang][driver] Try to support `flang -fc1as`

2023-02-08 Thread Shao-Ce SUN via Phabricator via cfe-commits
sunshaoce abandoned this revision. sunshaoce added a comment. A better implementation is in D143572 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143478/new/ https://reviews.llvm.org/D143478

[PATCH] D143478: [RFC][Flang][driver] Try to support `flang -fc1as`

2023-02-07 Thread Shao-Ce SUN via Phabricator via cfe-commits
sunshaoce updated this revision to Diff 495522. sunshaoce added a comment. Add a test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143478/new/ https://reviews.llvm.org/D143478 Files: clang/include/clang/Driver/Options.h

[PATCH] D143478: [RFC][Flang][driver] Try to support `flang -fc1as`

2023-02-07 Thread Shao-Ce SUN via Phabricator via cfe-commits
sunshaoce added a comment. In D143478#4109412 , @clementval wrote: > There should be some tests. I'm not sure how to check the generated binaries, can you provide a test case for reference? Thanks! Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D143478: [RFC][Flang][driver] Try to support `flang -fc1as`

2023-02-07 Thread Shao-Ce SUN via Phabricator via cfe-commits
sunshaoce added a comment. In D143478#4109479 , @awarzynski wrote: > Thanks for working on this! Before diving any deeper > > - Does LLVM Flang require `flang-new -fc1as`? > - If yes, why can't Flang and Clang share it? > > For better visibility, I

[PATCH] D143478: [RFC][Flang][driver] Try to support `flang -fc1as`

2023-02-07 Thread Andrzej Warzynski via Phabricator via cfe-commits
awarzynski added a comment. Thanks for working on this! Before diving any deeper - Does LLVM Flang require `flang-new -fc1as`? - If yes, why can't Flang and Clang share it? For better visibility, I suggest raising these questions on Discourse . As this could

[PATCH] D143478: [RFC][Flang][driver] Try to support `flang -fc1as`

2023-02-07 Thread Valentin Clement via Phabricator via cfe-commits
clementval added a comment. There should be some tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143478/new/ https://reviews.llvm.org/D143478 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D143478: [RFC][Flang][driver] Try to support `flang -fc1as`

2023-02-07 Thread Shao-Ce SUN via Phabricator via cfe-commits
sunshaoce created this revision. sunshaoce added reviewers: rovka, kiranchandramohan, Leporacanthicus, awarzynski, sscalpone, liaolucy. Herald added projects: Flang, All. sunshaoce requested review of this revision. Herald added subscribers: cfe-commits, jdoerfert, MaskRay. Herald added a