[PATCH] D132197: [RISCV] Use Triple::isRISCV/isRISCV32/isRISCV64 helps in some places. NFC
Philip Reames via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 19 07:22:09 PDT 2022
reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.
LGTM w/ optional comment.
================
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;
----------------
I would leave this one unchanged for consistency with surrounding code.
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