From 7948cd11d17bea2d8679b9898d9debdcec5c11be Mon Sep 17 00:00:00 2001 From: Fraser Cormack Date: Fri, 18 Dec 2020 21:49:14 +0000 Subject: [PATCH] [RISCV] Address clang-tidy warnings in RISCVTargetMachine. NFC. --- llvm/lib/Target/RISCV/RISCVTargetMachine.cpp | 12 +++++------- llvm/lib/Target/RISCV/RISCVTargetMachine.h | 2 +- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/llvm/lib/Target/RISCV/RISCVTargetMachine.cpp b/llvm/lib/Target/RISCV/RISCVTargetMachine.cpp index 5e1b623..fe35229 100644 --- a/llvm/lib/Target/RISCV/RISCVTargetMachine.cpp +++ b/llvm/lib/Target/RISCV/RISCVTargetMachine.cpp @@ -35,7 +35,7 @@ using namespace llvm; extern "C" LLVM_EXTERNAL_VISIBILITY void LLVMInitializeRISCVTarget() { RegisterTargetMachine X(getTheRISCV32Target()); RegisterTargetMachine Y(getTheRISCV64Target()); - auto PR = PassRegistry::getPassRegistry(); + auto *PR = PassRegistry::getPassRegistry(); initializeGlobalISel(*PR); initializeRISCVMergeBaseOffsetOptPass(*PR); initializeRISCVExpandPseudoPass(*PR); @@ -43,12 +43,10 @@ extern "C" LLVM_EXTERNAL_VISIBILITY void LLVMInitializeRISCVTarget() { } static StringRef computeDataLayout(const Triple &TT) { - if (TT.isArch64Bit()) { + if (TT.isArch64Bit()) return "e-m:e-p:64:64-i64:64-i128:128-n64-S128"; - } else { - assert(TT.isArch32Bit() && "only RV32 and RV64 are currently supported"); - return "e-m:e-p:32:32-i64:64-n32-S128"; - } + assert(TT.isArch32Bit() && "only RV32 and RV64 are currently supported"); + return "e-m:e-p:32:32-i64:64-n32-S128"; } static Reloc::Model getEffectiveRelocModel(const Triple &TT, @@ -143,7 +141,7 @@ public: void addPreSched2() override; void addPreRegAlloc() override; }; -} +} // namespace TargetPassConfig *RISCVTargetMachine::createPassConfig(PassManagerBase &PM) { return new RISCVPassConfig(*this, PM); diff --git a/llvm/lib/Target/RISCV/RISCVTargetMachine.h b/llvm/lib/Target/RISCV/RISCVTargetMachine.h index 9d1e04a..3156333 100644 --- a/llvm/lib/Target/RISCV/RISCVTargetMachine.h +++ b/llvm/lib/Target/RISCV/RISCVTargetMachine.h @@ -47,6 +47,6 @@ public: virtual bool isNoopAddrSpaceCast(unsigned SrcAS, unsigned DstAS) const override; }; -} +} // namespace llvm #endif -- 2.7.4