Skip to content

Commit 5a8fd65

Browse files
committedOct 30, 2018
[llvm-mca] Move namespace mca inside llvm::
Summary: This allows to remove `using namespace llvm;` in those *.cpp files When we want to revisit the decision (everything resides in llvm::mca::*) in the future, we can move things to a nested namespace of llvm::mca::, to conceptually make them separate from the rest of llvm::mca::* Reviewers: andreadb, mattd Reviewed By: andreadb Subscribers: javed.absar, tschuett, gbedwell, llvm-commits Differential Revision: https://reviews.llvm.org/D53407 llvm-svn: 345612
1 parent 7458344 commit 5a8fd65

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

59 files changed

+118
-42
lines changed
 

‎llvm/tools/llvm-mca/CodeRegion.cpp

+2
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414

1515
#include "CodeRegion.h"
1616

17+
namespace llvm {
1718
namespace mca {
1819

1920
bool CodeRegion::isLocInRange(llvm::SMLoc Loc) const {
@@ -63,3 +64,4 @@ void CodeRegions::addInstruction(const llvm::MCInst &Instruction) {
6364
}
6465

6566
} // namespace mca
67+
} // namespace llvm

‎llvm/tools/llvm-mca/CodeRegion.h

+2
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@
4141
#include "llvm/Support/SourceMgr.h"
4242
#include <vector>
4343

44+
namespace llvm {
4445
namespace mca {
4546

4647
/// A region of assembly code.
@@ -123,5 +124,6 @@ class CodeRegions {
123124
};
124125

125126
} // namespace mca
127+
} // namespace llvm
126128

127129
#endif

0 commit comments

Comments
 (0)