[llvm] c29d867 - [NFC] fix some clang-tidy warnings for hwasan pass
Florian Mayer via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 19 12:58:09 PST 2022
Author: Florian Mayer
Date: 2022-12-19T12:57:58-08:00
New Revision: c29d86783342273c43bc4ca7a2a011befd8e67c1
URL: https://github.com/llvm/llvm-project/commit/c29d86783342273c43bc4ca7a2a011befd8e67c1
DIFF: https://github.com/llvm/llvm-project/commit/c29d86783342273c43bc4ca7a2a011befd8e67c1.diff
LOG: [NFC] fix some clang-tidy warnings for hwasan pass
Added:
Modified:
llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp b/llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp
index 07b63c4bc3bae..d8d8680532964 100644
--- a/llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp
+++ b/llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp
@@ -494,11 +494,11 @@ void HWAddressSanitizer::createHwasanCtorComdat() {
Comdat *NoteComdat = M.getOrInsertComdat(kHwasanModuleCtorName);
Type *Int8Arr0Ty = ArrayType::get(Int8Ty, 0);
- auto Start =
+ auto *Start =
new GlobalVariable(M, Int8Arr0Ty, true, GlobalVariable::ExternalLinkage,
nullptr, "__start_hwasan_globals");
Start->setVisibility(GlobalValue::HiddenVisibility);
- auto Stop =
+ auto *Stop =
new GlobalVariable(M, Int8Arr0Ty, true, GlobalVariable::ExternalLinkage,
nullptr, "__stop_hwasan_globals");
Stop->setVisibility(GlobalValue::HiddenVisibility);
@@ -533,7 +533,7 @@ void HWAddressSanitizer::createHwasanCtorComdat() {
// Create a zero-length global in hwasan_globals so that the linker will
// always create start and stop symbols.
- auto Dummy = new GlobalVariable(
+ auto *Dummy = new GlobalVariable(
M, Int8Arr0Ty, /*isConstantGlobal*/ true, GlobalVariable::PrivateLinkage,
Constant::getNullValue(Int8Arr0Ty), "hwasan.dummy.global");
Dummy->setSection("hwasan_globals");
@@ -702,14 +702,13 @@ Value *HWAddressSanitizer::getShadowNonTls(IRBuilder<> &IRB) {
IRB, ConstantExpr::getIntToPtr(
ConstantInt::get(IntptrTy, Mapping.Offset), Int8PtrTy));
- if (Mapping.InGlobal) {
+ if (Mapping.InGlobal)
return getDynamicShadowIfunc(IRB);
- } else {
- Value *GlobalDynamicAddress =
- IRB.GetInsertBlock()->getParent()->getParent()->getOrInsertGlobal(
- kHwasanShadowMemoryDynamicAddress, Int8PtrTy);
- return IRB.CreateLoad(Int8PtrTy, GlobalDynamicAddress);
- }
+
+ Value *GlobalDynamicAddress =
+ IRB.GetInsertBlock()->getParent()->getParent()->getOrInsertGlobal(
+ kHwasanShadowMemoryDynamicAddress, Int8PtrTy);
+ return IRB.CreateLoad(Int8PtrTy, GlobalDynamicAddress);
}
bool HWAddressSanitizer::ignoreAccess(Instruction *Inst, Value *Ptr) {
@@ -766,7 +765,7 @@ void HWAddressSanitizer::getInterestingMemoryOperands(
Interesting.emplace_back(I, XCHG->getPointerOperandIndex(), true,
XCHG->getCompareOperand()->getType(),
std::nullopt);
- } else if (auto CI = dyn_cast<CallInst>(I)) {
+ } else if (auto *CI = dyn_cast<CallInst>(I)) {
for (unsigned ArgNo = 0; ArgNo < CI->arg_size(); ArgNo++) {
if (!ClInstrumentByval || !CI->isByValArgument(ArgNo) ||
ignoreAccess(I, CI->getArgOperand(ArgNo)))
@@ -1152,8 +1151,7 @@ Value *HWAddressSanitizer::getHwasanThreadSlotPtr(IRBuilder<> &IRB, Type *Ty) {
Value *HWAddressSanitizer::getPC(IRBuilder<> &IRB) {
if (TargetTriple.getArch() == Triple::aarch64)
return readRegister(IRB, "pc");
- else
- return IRB.CreatePtrToInt(IRB.GetInsertBlock()->getParent(), IntptrTy);
+ return IRB.CreatePtrToInt(IRB.GetInsertBlock()->getParent(), IntptrTy);
}
Value *HWAddressSanitizer::getSP(IRBuilder<> &IRB) {
@@ -1162,7 +1160,7 @@ Value *HWAddressSanitizer::getSP(IRBuilder<> &IRB) {
// first).
Function *F = IRB.GetInsertBlock()->getParent();
Module *M = F->getParent();
- auto GetStackPointerFn = Intrinsic::getDeclaration(
+ auto *GetStackPointerFn = Intrinsic::getDeclaration(
M, Intrinsic::frameaddress,
IRB.getInt8PtrTy(M->getDataLayout().getAllocaAddrSpace()));
CachedSP = IRB.CreatePtrToInt(
More information about the llvm-commits
mailing list