richard.barton.arm added inline comments.
================
Comment at: clang/lib/Driver/ToolChain.cpp:185
{"flang", "--driver-mode=flang"},
+ {"flang-new", "--driver-mode=flang"},
{"clang-dxc", "--driver-mode=dxc"},
----------------
clementval wrote:
> This is counter intuitive. We rename flang-new but we add flang-new here. It
> should be configurable.
This is a list of all the valid prefixes of binaries that the driver supports.
With this change, an additional one will be supported so I don't think it's an
issue to have both flang and flang-new here.
The thing that confuses me is how flang-new works today without this change,
given that "flang" is not a suffix of "flang-new"? @awarzynski , do you know
why that is? Perhaps the line here is not needed at all? Or is this a bug fix
for flang-new that is actually unrelated to this change?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125788/new/
https://reviews.llvm.org/D125788
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
