[PATCH] D132197: [RISCV] Use Triple::isRISCV/isRISCV32/isRISCV64 helps in some places. NFC
Craig Topper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 19 07:58:38 PDT 2022
craig.topper added inline comments.
================
Comment at: clang/lib/Driver/ToolChains/Linux.cpp:741
getTriple().getArch() == llvm::Triple::thumbeb;
- const bool IsRISCV64 = getTriple().getArch() == llvm::Triple::riscv64;
+ const bool IsRISCV64 = getTriple().isRISCV64();
const bool IsSystemZ = getTriple().getArch() == llvm::Triple::systemz;
----------------
reames wrote:
> I would leave this one unchanged for consistency with surrounding code.
It’s consistent with MIPS further up, but I can change it back.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132197/new/
https://reviews.llvm.org/D132197
More information about the cfe-commits
mailing list