[PATCH] D125788: [flang][driver] Rename `flang-new` as `flang`

2022-05-31 Thread Damian Rouson via Phabricator via cfe-commits
rouson added a comment. On Mon, May 30, 2022 at 2:39 AM Andrzej Warzynski via Phabricator wrote: > 1. Lets rename `flang` as `flang-to-external-fc` regardless of what's decided > for `flang-new`. That would already be a huge step forward (and > would > reflect accurately what the bash

[PATCH] D125788: [flang][driver] Rename `flang-new` as `flang`

2022-05-25 Thread Damian Rouson via Phabricator via cfe-commits
rouson added a comment. +1 @kiranchandramohan I have access to the NAG Fortran compiler and find it very helpful for checking standard conformance. I was unaware of the NAG test suite and don't see any mention of it online. Is it an official product that can be purchased? Repository: rG

[PATCH] D125788: [flang][driver] Rename `flang-new` as `flang`

2022-05-24 Thread Damian Rouson via Phabricator via cfe-commits
rouson added a comment. In D125788#3534482 , @sscalpone wrote: > My belief is that we should wait until f18 hits a reasonable quality bar for > executables before making flang-new the default. My reasoning seems not to > have carried the day. It

[PATCH] D125788: [flang][driver] Rename `flang-new` as `flang`

2022-05-18 Thread Damian Rouson via Phabricator via cfe-commits
rouson added a comment. I think this is an exciting step. I hope it gets approved. Although it's technically true that this could appear to be a regression for current flang script users, the ultimate compilation currently happens by invoking an external compiler so most current flang script

[PATCH] D122008: [flang][driver] Add support for generating executables

2022-04-13 Thread Damian Rouson via Phabricator via cfe-commits
rouson added a comment. @awarzynski I'll echo your comment that "... most people new to LLVM Flang use flang rather than flang-new and most (all?) find it very confusing." I've known of flang in various forms (including a flang predated classic flang) and yet I'm still finding new information

[PATCH] D122008: [flang][driver] Add support for generating executables

2022-04-13 Thread Damian Rouson via Phabricator via cfe-commits
rouson added a comment. @awarzynski based on @h-vetinari's optimism about adding flags, I still like option 2. If it's not possible to add flags, then I would prefer option 1. If we go with option 3, then I am at least encouraged by reviewing the upstreaming project status links that @kiran

[PATCH] D122008: [flang][driver] Add support for generating executables

2022-04-07 Thread Damian Rouson via Phabricator via cfe-commits
rouson added a comment. I also very much like the second option. I think it prevents a naive user from stumbling into treacherous territory. Also, as someone who is funded to write tests for flang, making it easier to build executables would expand the options for testing. Currently, we're