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

Tue Apr 19 01:56:58 PDT 2022

awarzynski added a comment.

In D122008#3456445 <https://reviews.llvm.org/D122008#3456445>, @clementval wrote:

> Do you plan to discuss this again during the next call? Note that today is a holiday in various country in Europe (maybe elsewhere too) so the one on 4/27 is probably better.

To be perfectly honest, I wasn't planning to.

The main reason to bring this up in one of our community calls was to increase the visibility of this patch. I feel that that goal has been achieved. Keeping the discussion here means that even people who are unable to join our calls can participate and share their view. As such, discussing on Phabricator is more inclusive.

While I appreciate that some people expressed preference for Option 3, it seemed to me that Option 2 is a compromise that will work for everyone. It hides the new functionality behind a flag and makes it very counter-intuitive to use. The flag itself makes it clear that this functionality is experimental. At the same time, Option 2 is sufficient to unblock progress in other areas.

If folks disagree and feel that Option 2 is still problematic, could you elaborate "why" and suggest an alternative? There's no need to wait for the next call to discuss this.

Thanks!

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122008/new/

https://reviews.llvm.org/D122008