[PATCH] D16028: If we don't recognise the target, explicitly specify the executable that can list targets.
Rafael EspĂndola via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 14 12:47:43 PST 2016
On 9 January 2016 at 20:22, Mehdi AMINI <mehdi.amini at apple.com> wrote:
> joker.eph added a subscriber: joker.eph.
>
> ================
> Comment at: lib/Support/TargetRegistry.cpp:77
> @@ -76,3 +76,3 @@
> Error = "No available targets are compatible with this triple, "
> - "see -version for the available targets.";
> + "see llc -version for the available targets.";
> return nullptr;
> ----------------
> While I agree that it is a problem to refer to a command line option in the library, it totally does not make sense to refer to `llc` here. I'd rather remove the references to command line flags entirely.
I agree.
Cheers,
Rafael
More information about the llvm-commits
mailing list