[NFC] Hide implementation details in anonymous namespaces
authorBenjamin Kramer <benny.kra@googlemail.com>
Sun, 8 Jan 2023 16:25:29 +0000 (17:25 +0100)
committerBenjamin Kramer <benny.kra@googlemail.com>
Sun, 8 Jan 2023 16:37:02 +0000 (17:37 +0100)
25 files changed:
clang/lib/AST/ASTImporter.cpp
clang/lib/Driver/OffloadBundler.cpp
clang/lib/Sema/AnalysisBasedWarnings.cpp
clang/lib/StaticAnalyzer/Checkers/RetainCountChecker/RetainCountDiagnostics.cpp
clang/lib/StaticAnalyzer/Core/BugReporterVisitors.cpp
clang/lib/StaticAnalyzer/Core/LoopUnrolling.cpp
llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp
llvm/lib/CodeGen/ExpandLargeDivRem.cpp
llvm/lib/CodeGen/ExpandLargeFpConvert.cpp
llvm/lib/CodeGen/MachineCycleAnalysis.cpp
llvm/lib/CodeGen/SanitizerBinaryMetadata.cpp
llvm/lib/MC/SPIRVObjectWriter.cpp
llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
llvm/lib/Target/AMDGPU/GCNVOPDUtils.cpp
llvm/lib/Target/AMDGPU/R600ISelDAGToDAG.cpp
llvm/lib/Target/AMDGPU/R600MCInstLower.cpp
llvm/lib/Target/AMDGPU/R600TargetMachine.cpp
llvm/lib/Target/RISCV/RISCVISelLowering.cpp
llvm/lib/Target/VE/VERegisterInfo.cpp
llvm/lib/Target/VE/VETargetMachine.cpp
llvm/lib/Target/X86/X86ReturnThunks.cpp
llvm/lib/Transforms/Scalar/LoopFlatten.cpp
llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp
llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
llvm/lib/Transforms/Vectorize/VectorCombine.cpp

index e77f570..329718c 100644 (file)
@@ -8683,6 +8683,7 @@ Expected<TypeSourceInfo *> ASTImporter::Import(TypeSourceInfo *FromTSI) {
   return ToContext.getTrivialTypeSourceInfo(*TOrErr, *BeginLocOrErr);
 }
 
+namespace {
 // To use this object, it should be created before the new attribute is created,
 // and destructed after it is created. The construction already performs the
 // import of the data.
@@ -8813,6 +8814,7 @@ public:
     return ToAttr;
   }
 };
+} // namespace
 
 Expected<Attr *> ASTImporter::Import(const Attr *FromAttr) {
   AttrImporter AI(*this);
index bcc0c32..86fc4df 100644 (file)
@@ -184,6 +184,7 @@ bool isCodeObjectCompatible(const OffloadTargetInfo &CodeObjectInfo,
   return true;
 }
 
+namespace {
 /// Generic file handler interface.
 class FileHandler {
 public:
@@ -491,8 +492,6 @@ public:
   }
 };
 
-namespace {
-
 // This class implements a list of temporary files that are removed upon
 // object destruction.
 class TempFileHandlerRAII {
@@ -524,8 +523,6 @@ private:
   std::forward_list<SmallString<128u>> Files;
 };
 
-} // end anonymous namespace
-
 /// Handler for object files. The bundles are organized by sections with a
 /// designated name.
 ///
@@ -827,6 +824,7 @@ public:
     return Error::success();
   }
 };
+} // namespace
 
 /// Return an appropriate object file handler. We use the specific object
 /// handler if we know how to deal with that format, otherwise we use a default
index bde2036..7f0677f 100644 (file)
@@ -2143,6 +2143,7 @@ public:
 // Unsafe buffer usage analysis.
 //===----------------------------------------------------------------------===//
 
+namespace {
 class UnsafeBufferUsageReporter : public UnsafeBufferUsageHandler {
   Sema &S;
 
@@ -2163,7 +2164,7 @@ public:
       D << F;
   }
 };
-
+} // namespace
 
 //===----------------------------------------------------------------------===//
 // AnalysisBasedWarnings - Worker object used by Sema to execute analysis-based
index 4583b21..afacf73 100644 (file)
@@ -613,6 +613,7 @@ static std::optional<std::string> describeRegion(const MemRegion *MR) {
 
 using Bindings = llvm::SmallVector<std::pair<const MemRegion *, SVal>, 4>;
 
+namespace {
 class VarBindingsCollector : public StoreManager::BindingsHandler {
   SymbolRef Sym;
   Bindings &Result;
@@ -633,6 +634,7 @@ public:
     return true;
   }
 };
+} // namespace
 
 Bindings getAllVarBindingsForSymbol(ProgramStateManager &Manager,
                                     const ExplodedNode *Node, SymbolRef Sym) {
index eecf049..01ba95a 100644 (file)
@@ -1186,8 +1186,6 @@ public:
   }
 };
 
-} // end of anonymous namespace
-
 //===----------------------------------------------------------------------===//
 //                               StoreSiteFinder
 //===----------------------------------------------------------------------===//
@@ -1227,6 +1225,7 @@ public:
                                    BugReporterContext &BRC,
                                    PathSensitiveBugReport &BR) override;
 };
+} // namespace
 
 void StoreSiteFinder::Profile(llvm::FoldingSetNodeID &ID) const {
   static int tag = 0;
@@ -2216,6 +2215,7 @@ PathDiagnosticPieceRef StoreHandler::constructNote(StoreInfo SI,
   return std::make_shared<PathDiagnosticEventPiece>(L, NodeText);
 }
 
+namespace {
 class DefaultStoreHandler final : public StoreHandler {
 public:
   using StoreHandler::StoreHandler;
@@ -2600,6 +2600,7 @@ public:
     return CombinedResult;
   }
 };
+} // namespace
 
 Tracker::Tracker(PathSensitiveBugReport &Report) : Report(Report) {
   // Default expression handlers.
index f6efde8..69599d1 100644 (file)
@@ -24,6 +24,7 @@ using namespace clang::ast_matchers;
 
 static const int MAXIMUM_STEP_UNROLLED = 128;
 
+namespace {
 struct LoopState {
 private:
   enum Kind { Normal, Unrolled } K;
@@ -56,6 +57,7 @@ public:
     ID.AddInteger(maxStep);
   }
 };
+} // namespace
 
 // The tracked stack of loops. The stack indicates that which loops the
 // simulated element contained by. The loops are marked depending if we decided
index 294f5f6..2049ada 100644 (file)
@@ -278,6 +278,7 @@ static DebugAggregate getAggregate(const DebugVariable &Var) {
   return DebugAggregate(Var.getVariable(), Var.getInlinedAt());
 }
 
+namespace {
 /// In dwarf emission, the following sequence
 ///    1. dbg.value ... Fragment(0, 64)
 ///    2. dbg.value ... Fragment(0, 32)
@@ -1121,6 +1122,7 @@ public:
   /// true if any variable locations have been added to FnVarLocs.
   bool run(FunctionVarLocsBuilder *FnVarLocs);
 };
+} // namespace
 
 void AssignmentTrackingLowering::setLocKind(BlockInfo *LiveSet, VariableID Var,
                                             LocKind K) {
index f8ffc2b..057b531 100644 (file)
@@ -105,6 +105,7 @@ static bool runImpl(Function &F, const TargetLowering &TLI) {
   return Modified;
 }
 
+namespace {
 class ExpandLargeDivRemLegacyPass : public FunctionPass {
 public:
   static char ID;
@@ -125,6 +126,7 @@ public:
     AU.addPreserved<GlobalsAAWrapperPass>();
   }
 };
+} // namespace
 
 char ExpandLargeDivRemLegacyPass::ID = 0;
 INITIALIZE_PASS_BEGIN(ExpandLargeDivRemLegacyPass, "expand-large-div-rem",
index c21fbc1..ca8056a 100644 (file)
@@ -629,6 +629,7 @@ static bool runImpl(Function &F, const TargetLowering &TLI) {
   return Modified;
 }
 
+namespace {
 class ExpandLargeFpConvertLegacyPass : public FunctionPass {
 public:
   static char ID;
@@ -650,6 +651,7 @@ public:
     AU.addPreserved<GlobalsAAWrapperPass>();
   }
 };
+} // namespace
 
 char ExpandLargeFpConvertLegacyPass::ID = 0;
 INITIALIZE_PASS_BEGIN(ExpandLargeFpConvertLegacyPass, "expand-large-fp-convert",
index 8a6030c..0a59aba 100644 (file)
@@ -54,6 +54,7 @@ void MachineCycleInfoWrapperPass::releaseMemory() {
   F = nullptr;
 }
 
+namespace {
 class MachineCycleInfoPrinterPass : public MachineFunctionPass {
 public:
   static char ID;
@@ -63,6 +64,7 @@ public:
   bool runOnMachineFunction(MachineFunction &F) override;
   void getAnalysisUsage(AnalysisUsage &AU) const override;
 };
+} // namespace
 
 char MachineCycleInfoPrinterPass::ID = 0;
 
index f9e608f..dd70a2f 100644 (file)
@@ -24,6 +24,7 @@
 
 using namespace llvm;
 
+namespace {
 class MachineSanitizerBinaryMetadata : public MachineFunctionPass {
 public:
   static char ID;
@@ -31,6 +32,7 @@ public:
   MachineSanitizerBinaryMetadata();
   bool runOnMachineFunction(MachineFunction &F) override;
 };
+} // namespace
 
 INITIALIZE_PASS(MachineSanitizerBinaryMetadata, "machine-sanmd",
                 "Machine Sanitizer Binary Metadata", false, false)
index 4a07740..cb49f5e 100644 (file)
@@ -14,6 +14,7 @@
 
 using namespace llvm;
 
+namespace {
 class SPIRVObjectWriter : public MCObjectWriter {
   ::support::endian::Writer W;
 
@@ -38,6 +39,7 @@ private:
   uint64_t writeObject(MCAssembler &Asm, const MCAsmLayout &Layout) override;
   void writeHeader(const MCAssembler &Asm);
 };
+} // namespace
 
 void SPIRVObjectWriter::writeHeader(const MCAssembler &Asm) {
   constexpr uint32_t MagicNumber = 0x07230203;
index f5f6c07..9f18159 100644 (file)
@@ -39,6 +39,7 @@ static cl::opt<unsigned> SVEGatherOverhead("sve-gather-overhead", cl::init(10),
 static cl::opt<unsigned> SVEScatterOverhead("sve-scatter-overhead",
                                             cl::init(10), cl::Hidden);
 
+namespace {
 class TailFoldingKind {
 private:
   uint8_t Bits = 0; // Currently defaults to disabled.
@@ -89,6 +90,7 @@ public:
   void add(uint8_t Flag) { Bits |= Flag; }
   void remove(uint8_t Flag) { Bits &= ~Flag; }
 };
+} // namespace
 
 TailFoldingKind TailFoldingKindLoc;
 
index 2b203a5..95ea422 100644 (file)
@@ -136,6 +136,7 @@ static bool shouldScheduleVOPDAdjacent(const TargetInstrInfo &TII,
   return checkVOPDRegConstraints(STII, *FirstMI, SecondMI);
 }
 
+namespace {
 /// Adapts design from MacroFusion
 /// Puts valid candidate instructions back-to-back so they can easily
 /// be turned into VOPD instructions
@@ -177,6 +178,7 @@ struct VOPDPairingMutation : ScheduleDAGMutation {
     LLVM_DEBUG(dbgs() << "Completed VOPDPairingMutation\n");
   }
 };
+} // namespace
 
 std::unique_ptr<ScheduleDAGMutation> llvm::createVOPDPairingMutation() {
   return std::make_unique<VOPDPairingMutation>(shouldScheduleVOPDAdjacent);
index 7d2fdf5..50a90dd 100644 (file)
@@ -18,6 +18,7 @@
 #include "R600Subtarget.h"
 #include "llvm/Analysis/ValueTracking.h"
 
+namespace {
 class R600DAGToDAGISel : public AMDGPUDAGToDAGISel {
   const R600Subtarget *Subtarget;
 
@@ -47,6 +48,7 @@ protected:
   // Include the pieces autogenerated from the target description.
 #include "R600GenDAGISel.inc"
 };
+} // namespace
 
 bool R600DAGToDAGISel::runOnMachineFunction(MachineFunction &MF) {
   Subtarget = &MF.getSubtarget<R600Subtarget>();
index f81f512..6da5456 100644 (file)
@@ -20,6 +20,7 @@
 #include "llvm/MC/MCContext.h"
 #include "llvm/MC/MCExpr.h"
 
+namespace {
 class R600MCInstLower : public AMDGPUMCInstLower {
 public:
   R600MCInstLower(MCContext &ctx, const R600Subtarget &ST,
@@ -28,6 +29,7 @@ public:
   /// Lower a MachineInstr to an MCInst
   void lower(const MachineInstr *MI, MCInst &OutMI) const;
 };
+} // namespace
 
 R600MCInstLower::R600MCInstLower(MCContext &Ctx, const R600Subtarget &ST,
                                  const AsmPrinter &AP)
index 506e7b8..3684058 100644 (file)
@@ -88,6 +88,7 @@ R600TargetMachine::getTargetTransformInfo(const Function &F) const {
   return TargetTransformInfo(R600TTIImpl(this, F));
 }
 
+namespace {
 class R600PassConfig final : public AMDGPUPassConfig {
 public:
   R600PassConfig(LLVMTargetMachine &TM, PassManagerBase &PM)
@@ -104,6 +105,7 @@ public:
   void addPreSched2() override;
   void addPreEmitPass() override;
 };
+} // namespace
 
 //===----------------------------------------------------------------------===//
 // R600 Pass Setup
index db1bd64..f946225 100644 (file)
@@ -8509,6 +8509,7 @@ static SDValue performTRUNCATECombine(SDNode *N, SelectionDAG &DAG,
   return SDValue();
 }
 
+namespace {
 // Helper class contains information about comparison operation.
 // The first two operands of this operation are compared values and the
 // last one is the operation.
@@ -8602,6 +8603,7 @@ public:
   // Returns consition code of comparison operation.
   ISD::CondCode getCondCode() const { return CCode; }
 };
+} // namespace
 
 // Verifies conditions to apply an optimization.
 // Returns Reference comparison code and three operands A, B, C.
index dfdddf0..cd3ccd9 100644 (file)
@@ -126,6 +126,7 @@ static unsigned offsetToDisp(MachineInstr &MI) {
   return OffDisp;
 }
 
+namespace {
 class EliminateFrameIndex {
   const TargetInstrInfo &TII;
   const TargetRegisterInfo &TRI;
@@ -192,6 +193,7 @@ public:
   void processMI(MachineInstr &MI, Register FrameReg, int64_t Offset,
                  int FIOperandNum);
 };
+} // namespace
 
 // Prepare the frame index if it doesn't fit in the immediate field.  Use
 // clobber register to hold calculated address.
index f12bd0f..93fb3d8 100644 (file)
@@ -69,12 +69,14 @@ static Reloc::Model getEffectiveRelocModel(std::optional<Reloc::Model> RM) {
   return RM.value_or(Reloc::Static);
 }
 
+namespace {
 class VEELFTargetObjectFile : public TargetLoweringObjectFileELF {
   void Initialize(MCContext &Ctx, const TargetMachine &TM) override {
     TargetLoweringObjectFileELF::Initialize(Ctx, TM);
     InitializeELF(TM.Options.UseInitArray);
   }
 };
+} // namespace
 
 static std::unique_ptr<TargetLoweringObjectFile> createTLOF() {
   return std::make_unique<VEELFTargetObjectFile>();
index 0107fab..aaa2045 100644 (file)
@@ -43,12 +43,14 @@ using namespace llvm;
 #define PASS_KEY "x86-return-thunks"
 #define DEBUG_TYPE PASS_KEY
 
+namespace {
 struct X86ReturnThunks final : public MachineFunctionPass {
   static char ID;
   X86ReturnThunks() : MachineFunctionPass(ID) {}
   StringRef getPassName() const override { return "X86 Return Thunks"; }
   bool runOnMachineFunction(MachineFunction &MF) override;
 };
+} // namespace
 
 char X86ReturnThunks::ID = 0;
 
index d82624e..5a43d37 100644 (file)
@@ -100,6 +100,7 @@ static cl::opt<bool>
             cl::desc("Widen the loop induction variables, if possible, so "
                      "overflow checks won't reject flattening"));
 
+namespace {
 // We require all uses of both induction variables to match this pattern:
 //
 //   (OuterPHI * InnerTripCount) + InnerPHI
@@ -288,6 +289,7 @@ struct FlattenInfo {
     return true;
   }
 };
+} // namespace
 
 static bool
 setLoopComponents(Value *&TC, Value *&TripCount, BinaryOperator *&Increment,
index 85e2bef..2dc97a4 100644 (file)
@@ -1272,6 +1272,7 @@ bool LoopIdiomRecognize::processLoopStoreOfLoopLoad(StoreInst *SI,
                                     StoreEv, LoadEv, BECount);
 }
 
+namespace {
 class MemmoveVerifier {
 public:
   explicit MemmoveVerifier(const Value &LoadBasePtr, const Value &StoreBasePtr,
@@ -1315,6 +1316,7 @@ private:
 public:
   const bool IsSameObject;
 };
+} // namespace
 
 bool LoopIdiomRecognize::processLoopStoreOfLoopLoad(
     Value *DestPtr, Value *SourcePtr, const SCEV *StoreSizeSCEV,
index c37c63c..6f8550b 100644 (file)
@@ -419,8 +419,10 @@ static std::optional<unsigned> getSmallBestKnownTC(ScalarEvolution &SE,
   return std::nullopt;
 }
 
+namespace {
 // Forward declare GeneratedRTChecks.
 class GeneratedRTChecks;
+} // namespace
 
 namespace llvm {
 
@@ -1867,6 +1869,7 @@ public:
 };
 } // end namespace llvm
 
+namespace {
 /// Helper struct to manage generating runtime checks for vectorization.
 ///
 /// The runtime checks are created up-front in temporary blocks to allow better
@@ -2131,6 +2134,7 @@ public:
     return MemCheckBlock;
   }
 };
+} // namespace
 
 // Return true if \p OuterLp is an outer loop annotated with hints for explicit
 // vectorization. The loop needs to be annotated with #pragma omp simd
index 9dec820..455fb39 100644 (file)
@@ -947,6 +947,7 @@ static bool isMemModifiedBetween(BasicBlock::iterator Begin,
   });
 }
 
+namespace {
 /// Helper class to indicate whether a vector index can be safely scalarized and
 /// if a freeze needs to be inserted.
 class ScalarizationResult {
@@ -1001,6 +1002,7 @@ public:
     ToFreeze = nullptr;
   }
 };
+} // namespace
 
 /// Check if it is legal to scalarize a memory access to \p VecTy at index \p
 /// Idx. \p Idx must access a valid vector element.