diff options
author | Craig Topper <craig.topper@intel.com> | 2018-03-07 17:53:16 +0000 |
---|---|---|
committer | Craig Topper <craig.topper@intel.com> | 2018-03-07 17:53:16 +0000 |
commit | 8665f59e2ccce18a6775623b51d0b4d201ec3cb2 (patch) | |
tree | d0bf22f035cb8d40dae8dacefc579c538d9bd40d | |
parent | 2c3edf05679e3c8cf341b2d1b2f517533cdae6e3 (diff) | |
download | bcm5719-llvm-8665f59e2ccce18a6775623b51d0b4d201ec3cb2.tar.gz bcm5719-llvm-8665f59e2ccce18a6775623b51d0b4d201ec3cb2.zip |
[Support] Stop passing StringRefs by const reference in some of the getHostCPUname implementations. NFC
llvm-svn: 326916
-rw-r--r-- | llvm/include/llvm/Support/Host.h | 6 | ||||
-rw-r--r-- | llvm/lib/Support/Host.cpp | 15 |
2 files changed, 9 insertions, 12 deletions
diff --git a/llvm/include/llvm/Support/Host.h b/llvm/include/llvm/Support/Host.h index a4b0a340c56..ddc5fa5796f 100644 --- a/llvm/include/llvm/Support/Host.h +++ b/llvm/include/llvm/Support/Host.h @@ -88,9 +88,9 @@ constexpr bool IsBigEndianHost = false; namespace detail { /// Helper functions to extract HostCPUName from /proc/cpuinfo on linux. - StringRef getHostCPUNameForPowerPC(const StringRef &ProcCpuinfoContent); - StringRef getHostCPUNameForARM(const StringRef &ProcCpuinfoContent); - StringRef getHostCPUNameForS390x(const StringRef &ProcCpuinfoContent); + StringRef getHostCPUNameForPowerPC(StringRef ProcCpuinfoContent); + StringRef getHostCPUNameForARM(StringRef ProcCpuinfoContent); + StringRef getHostCPUNameForS390x(StringRef ProcCpuinfoContent); StringRef getHostCPUNameForBPF(); } } diff --git a/llvm/lib/Support/Host.cpp b/llvm/lib/Support/Host.cpp index 0f3be408fa6..656e2577608 100644 --- a/llvm/lib/Support/Host.cpp +++ b/llvm/lib/Support/Host.cpp @@ -65,8 +65,7 @@ static std::unique_ptr<llvm::MemoryBuffer> return std::move(*Text); } -StringRef sys::detail::getHostCPUNameForPowerPC( - const StringRef &ProcCpuinfoContent) { +StringRef sys::detail::getHostCPUNameForPowerPC(StringRef ProcCpuinfoContent) { // Access to the Processor Version Register (PVR) on PowerPC is privileged, // and so we must use an operating-system interface to determine the current // processor type. On Linux, this is exposed through the /proc/cpuinfo file. @@ -145,8 +144,7 @@ StringRef sys::detail::getHostCPUNameForPowerPC( .Default(generic); } -StringRef sys::detail::getHostCPUNameForARM( - const StringRef &ProcCpuinfoContent) { +StringRef sys::detail::getHostCPUNameForARM(StringRef ProcCpuinfoContent) { // The cpuid register on arm is not accessible from user space. On Linux, // it is exposed through the /proc/cpuinfo file. @@ -250,8 +248,7 @@ StringRef sys::detail::getHostCPUNameForARM( return "generic"; } -StringRef sys::detail::getHostCPUNameForS390x( - const StringRef &ProcCpuinfoContent) { +StringRef sys::detail::getHostCPUNameForS390x(StringRef ProcCpuinfoContent) { // STIDP is a privileged operation, so use /proc/cpuinfo instead. // The "processor 0:" line comes after a fair amount of other information, @@ -1062,19 +1059,19 @@ StringRef sys::getHostCPUName() { #elif defined(__linux__) && (defined(__ppc__) || defined(__powerpc__)) StringRef sys::getHostCPUName() { std::unique_ptr<llvm::MemoryBuffer> P = getProcCpuinfoContent(); - const StringRef& Content = P ? P->getBuffer() : ""; + StringRef Content = P ? P->getBuffer() : ""; return detail::getHostCPUNameForPowerPC(Content); } #elif defined(__linux__) && (defined(__arm__) || defined(__aarch64__)) StringRef sys::getHostCPUName() { std::unique_ptr<llvm::MemoryBuffer> P = getProcCpuinfoContent(); - const StringRef& Content = P ? P->getBuffer() : ""; + StringRef Content = P ? P->getBuffer() : ""; return detail::getHostCPUNameForARM(Content); } #elif defined(__linux__) && defined(__s390x__) StringRef sys::getHostCPUName() { std::unique_ptr<llvm::MemoryBuffer> P = getProcCpuinfoContent(); - const StringRef& Content = P ? P->getBuffer() : ""; + StringRef Content = P ? P->getBuffer() : ""; return detail::getHostCPUNameForS390x(Content); } #else |