[PATCH] D109727: [Driver] Remove unneeded *-suse-* triples
Luís Marques via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 16 04:36:58 PDT 2021
luismarques added a comment.
Regarding D74399 <https://reviews.llvm.org/D74399>, cmake on a fedora RISC-V host still detects a generic triple:
-- LLVM host triple: riscv64-unknown-linux-gnu
-- LLVM default target triple: riscv64-unknown-linux-gnu
as opposed to the `gcc -dumpmachine` triple of `riscv64-redhat-linux` (with the vendor and no `-gnu` suffix), needed for a working toolchain. (I don't know if that cmake behaviour is uncommon or not).
Are you saying that it's fine for D74399 <https://reviews.llvm.org/D74399> to remain because of that, or does your argument also imply that D74399 <https://reviews.llvm.org/D74399> should be reverted? Would the answer change if cmake started detecting the "proper" triple? (and what would the deprecation path for that be, if any?).
================
Comment at: clang/lib/Driver/ToolChains/Gnu.cpp:2108-2113
+ "x86_64-linux-gnu", "x86_64-unknown-linux-gnu",
+ "x86_64-pc-linux-gnu", "x86_64-redhat-linux6E",
+ "x86_64-redhat-linux", "x86_64-manbo-linux-gnu",
+ "x86_64-linux-gnu", "x86_64-slackware-linux",
+ "x86_64-unknown-linux", "x86_64-amazon-linux",
+ "x86_64-linux-android"};
----------------
Ugh, these lines with multiple entries make mentally parsing the diff rather annoying.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109727/new/
https://reviews.llvm.org/D109727
More information about the cfe-commits
mailing list