diff options
| author | Jason Henline <jhen@google.com> | 2016-09-13 19:25:43 +0000 |
|---|---|---|
| committer | Jason Henline <jhen@google.com> | 2016-09-13 19:25:43 +0000 |
| commit | fb621479491f353562ec745a3daa402123110615 (patch) | |
| tree | d056f6065c45f1d2ea316a6a3ebfe44b81d1e54a /parallel-libs/streamexecutor/lib/KernelSpec.cpp | |
| parent | 6956d290341b3ceb7fb7599c9c772c08b0c2450a (diff) | |
| download | bcm5719-llvm-fb621479491f353562ec745a3daa402123110615.tar.gz bcm5719-llvm-fb621479491f353562ec745a3daa402123110615.zip | |
[SE] Add .clang-format
Summary:
The .clang-tidy file is copied from the top-level LLVM source directory.
Also fix warnings generated by clang-format:
* Moved SimpleHostPlatformDevice.h so its header include guard could
have the right format.
* Changed signatures of methods taking llvm::Twine by value to take it
by const ref instead.
* Add "noexcept" to some move constructors and assignment operators.
* Removed a bunch of places where single-statement loops and
conditionals were surrounded with braces. (This was not found by the
current clang-tidy, but with a local patch that I hope to upstream
soon.)
Reviewers: jlebar, jprice
Subscribers: parallel_libs-commits
Differential Revision: https://reviews.llvm.org/D24468
llvm-svn: 281374
Diffstat (limited to 'parallel-libs/streamexecutor/lib/KernelSpec.cpp')
| -rw-r--r-- | parallel-libs/streamexecutor/lib/KernelSpec.cpp | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/parallel-libs/streamexecutor/lib/KernelSpec.cpp b/parallel-libs/streamexecutor/lib/KernelSpec.cpp index d2715aa88a5..b5753a489d1 100644 --- a/parallel-libs/streamexecutor/lib/KernelSpec.cpp +++ b/parallel-libs/streamexecutor/lib/KernelSpec.cpp @@ -25,9 +25,8 @@ CUDAPTXInMemorySpec::CUDAPTXInMemorySpec( llvm::StringRef KernelName, const llvm::ArrayRef<CUDAPTXInMemorySpec::PTXSpec> SpecList) : KernelLoaderSpec(KernelName) { - for (const auto &Spec : SpecList) { + for (const auto &Spec : SpecList) PTXByComputeCapability.emplace(Spec.TheComputeCapability, Spec.PTXCode); - } } const char *CUDAPTXInMemorySpec::getCode(int ComputeCapabilityMajor, @@ -35,9 +34,8 @@ const char *CUDAPTXInMemorySpec::getCode(int ComputeCapabilityMajor, auto PTXIter = PTXByComputeCapability.find(CUDAPTXInMemorySpec::ComputeCapability{ ComputeCapabilityMajor, ComputeCapabilityMinor}); - if (PTXIter == PTXByComputeCapability.end()) { + if (PTXIter == PTXByComputeCapability.end()) return nullptr; - } return PTXIter->second; } @@ -50,12 +48,11 @@ OpenCLTextInMemorySpec::OpenCLTextInMemorySpec(llvm::StringRef KernelName, : KernelLoaderSpec(KernelName), Text(Text) {} void MultiKernelLoaderSpec::setKernelName(llvm::StringRef KernelName) { - if (TheKernelName) { + if (TheKernelName) assert(KernelName.equals(*TheKernelName) && "different kernel names in one MultiKernelLoaderSpec"); - } else { + else TheKernelName = llvm::make_unique<std::string>(KernelName); - } } MultiKernelLoaderSpec &MultiKernelLoaderSpec::addCUDAPTXInMemory( |

