Skip to content

Commit c5b5451

Browse files
[llvm-exegesis] Remove redundant declarations (NFC) (#166044)
In C++17, static constexpr members are implicitly inline, so they no longer require an out-of-line definition. This patch removes the unnecessary "const" from the in-class declaration as well. Identified with readability-redundant-declaration.
1 parent c9ef3d8 commit c5b5451

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

llvm/tools/llvm-exegesis/lib/ParallelSnippetGenerator.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -350,7 +350,5 @@ ParallelSnippetGenerator::generateCodeTemplates(
350350
return Result;
351351
}
352352

353-
constexpr const size_t ParallelSnippetGenerator::kMinNumDifferentAddresses;
354-
355353
} // namespace exegesis
356354
} // namespace llvm

llvm/tools/llvm-exegesis/lib/ParallelSnippetGenerator.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class ParallelSnippetGenerator : public SnippetGenerator {
2828
generateCodeTemplates(InstructionTemplate Variant,
2929
const BitVector &ForbiddenRegisters) const override;
3030

31-
static constexpr const size_t kMinNumDifferentAddresses = 6;
31+
static constexpr size_t kMinNumDifferentAddresses = 6;
3232

3333
private:
3434
// Instantiates memory operands within a snippet.

0 commit comments

Comments
 (0)