[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