[Attributor] Fix qualifier warning of the unittest

[bbn]

Summary

This patch fixes the warning of casting away qualifiers.

[kuter]

llvm/unittests/Transforms/IPO/AttributorTest.cpp

46–47

I think this should be a C++ cast.

[bbn]

use C++ cast

[jdoerfert]

Why do you need the cast anyway:

const AbstractAttribute *AA = &A.getOrCreateAAFor<AAIsDead>(IRPosition::function(*F))

should do fine, does it not?

[bbn]

remove the cast.

[jdoerfert]

LGTM

llvm/unittests/Transforms/IPO/AttributorTest.cpp

for (Function &F : M)
     Functions.insert(&F);
 
   CallGraphUpdater CGUpdater;
   BumpPtrAllocator Allocator;
   InformationCache InfoCache(M, AG, Allocator, nullptr);
   Attributor A(Functions, InfoCache, CGUpdater);
 
   Function *F = M.getFunction("foo");
 
-  AbstractAttribute *AA = (AbstractAttribute *)&(
-      A.getOrCreateAAFor<AAIsDead>(IRPosition::function(*F)));
+  const AbstractAttribute *AA =
+      &A.getOrCreateAAFor<AAIsDead>(IRPosition::function(*F));
 
   EXPECT_TRUE(AA);
 
   const auto *SFail = dyn_cast<AAAlign>(AA);
   const auto *SSucc = dyn_cast<AAIsDead>(AA);
 
   ASSERT_EQ(SFail, nullptr);
   ASSERT_TRUE(SSucc);
 }
 
 } // namespace llvm
 No newline at end of file