Take over and fix comments

Author: shiltian

llvm/include/llvm/Transforms/IPO/Attributor.h

@@ -1448,7 +1448,7 @@ struct AttributorConfig {
   /// Callback function to determine if an indirect call targets should be made
   /// direct call targets (with an if-cascade).
   std::function<bool(Attributor &A, const AbstractAttribute &AA, CallBase &CB,
-                     Function &AssummedCallee, unsigned NumCallees)>
+                     Function &AssumedCallee, unsigned NumCallees)>
       IndirectCalleeSpecializationCallback = nullptr;
 
   /// Helper to update an underlying call graph and to delete functions.

llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp

@@ -1064,7 +1064,6 @@ static bool runImpl(Module &M, AnalysisGetter &AG, TargetMachine &TM,
   AC.Allowed = &Allowed;
   AC.IsModulePass = true;
   AC.DefaultInitializeLiveInternals = false;
-  errs() << "HasWholeProgramVisibility " << HasWholeProgramVisibility << "\n";
   AC.IsClosedWorldModule = HasWholeProgramVisibility;
   AC.IndirectCalleeSpecializationCallback =
       IndirectCalleeSpecializationCallback;

llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp

@@ -738,7 +738,7 @@ void AMDGPUTargetMachine::registerPassBuilderCallbacks(PassBuilder &PB) {
       [this](ModulePassManager &PM, OptimizationLevel Level) {
         if (Level != OptimizationLevel::O0)
           PM.addPass(
-              AMDGPUAttributorPass(*this, /*HasWholeProgramVisibility*/ true));
+              AMDGPUAttributorPass(*this, /*HasWholeProgramVisibility=*/true));
         // We want to support the -lto-partitions=N option as "best effort".
         // For that, we need to lower LDS earlier in the pipeline before the
         // module is partitioned for codegen.