[llvm] r341541 - [MSan] revert r341528 to unbreak the bots
Alexander Potapenko via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 6 05:19:27 PDT 2018
Author: glider
Date: Thu Sep 6 05:19:27 2018
New Revision: 341541
URL: http://llvm.org/viewvc/llvm-project?rev=341541&view=rev
Log:
[MSan] revert r341528 to unbreak the bots
Removed:
llvm/trunk/test/Instrumentation/MemorySanitizer/X86/vararg_call.ll
Modified:
llvm/trunk/lib/Transforms/Instrumentation/MemorySanitizer.cpp
Modified: llvm/trunk/lib/Transforms/Instrumentation/MemorySanitizer.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Instrumentation/MemorySanitizer.cpp?rev=341541&r1=341540&r2=341541&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Instrumentation/MemorySanitizer.cpp (original)
+++ llvm/trunk/lib/Transforms/Instrumentation/MemorySanitizer.cpp Thu Sep 6 05:19:27 2018
@@ -448,10 +448,6 @@ private:
/// parameters (x86_64-specific).
GlobalVariable *VAArgTLS;
- /// Thread-local shadow storage for in-register va_arg function
- /// parameters (x86_64-specific).
- GlobalVariable *VAArgOriginTLS;
-
/// Thread-local shadow storage for va_arg overflow area
/// (x86_64-specific).
GlobalVariable *VAArgOverflowSizeTLS;
@@ -564,12 +560,6 @@ void MemorySanitizer::createUserspaceApi
M, ArrayType::get(IRB.getInt64Ty(), kParamTLSSize / 8), false,
GlobalVariable::ExternalLinkage, nullptr, "__msan_va_arg_tls", nullptr,
GlobalVariable::InitialExecTLSModel);
-
- VAArgOriginTLS = new GlobalVariable(
- M, ArrayType::get(OriginTy, kParamTLSSize / 4), false,
- GlobalVariable::ExternalLinkage, nullptr, "__msan_va_arg_origin_tls",
- nullptr, GlobalVariable::InitialExecTLSModel);
-
VAArgOverflowSizeTLS = new GlobalVariable(
M, IRB.getInt64Ty(), false, GlobalVariable::ExternalLinkage, nullptr,
"__msan_va_arg_overflow_size_tls", nullptr,
@@ -3268,7 +3258,6 @@ struct VarArgAMD64Helper : public VarArg
MemorySanitizer &MS;
MemorySanitizerVisitor &MSV;
Value *VAArgTLSCopy = nullptr;
- Value *VAArgTLSOriginCopy = nullptr;
Value *VAArgOverflowSize = nullptr;
SmallVector<CallInst*, 16> VAStartInstrumentationList;
@@ -3331,9 +3320,6 @@ struct VarArgAMD64Helper : public VarArg
uint64_t ArgSize = DL.getTypeAllocSize(RealTy);
Value *ShadowBase = getShadowPtrForVAArgument(
RealTy, IRB, OverflowOffset, alignTo(ArgSize, 8));
- Value *OriginBase = nullptr;
- if (MS.TrackOrigins)
- OriginBase = getOriginPtrForVAArgument(RealTy, IRB, OverflowOffset);
OverflowOffset += alignTo(ArgSize, 8);
if (!ShadowBase)
continue;
@@ -3344,31 +3330,22 @@ struct VarArgAMD64Helper : public VarArg
IRB.CreateMemCpy(ShadowBase, kShadowTLSAlignment, ShadowPtr,
kShadowTLSAlignment, ArgSize);
- if (MS.TrackOrigins)
- IRB.CreateMemCpy(OriginBase, kShadowTLSAlignment, OriginPtr,
- kShadowTLSAlignment, ArgSize);
} else {
ArgKind AK = classifyArgument(A);
if (AK == AK_GeneralPurpose && GpOffset >= AMD64GpEndOffset)
AK = AK_Memory;
if (AK == AK_FloatingPoint && FpOffset >= AMD64FpEndOffset)
AK = AK_Memory;
- Value *ShadowBase, *OriginBase = nullptr;
+ Value *ShadowBase;
switch (AK) {
case AK_GeneralPurpose:
ShadowBase =
getShadowPtrForVAArgument(A->getType(), IRB, GpOffset, 8);
- if (MS.TrackOrigins)
- OriginBase =
- getOriginPtrForVAArgument(A->getType(), IRB, GpOffset);
GpOffset += 8;
break;
case AK_FloatingPoint:
ShadowBase =
getShadowPtrForVAArgument(A->getType(), IRB, FpOffset, 16);
- if (MS.TrackOrigins)
- OriginBase =
- getOriginPtrForVAArgument(A->getType(), IRB, FpOffset);
FpOffset += 16;
break;
case AK_Memory:
@@ -3377,26 +3354,16 @@ struct VarArgAMD64Helper : public VarArg
uint64_t ArgSize = DL.getTypeAllocSize(A->getType());
ShadowBase =
getShadowPtrForVAArgument(A->getType(), IRB, OverflowOffset, 8);
- if (MS.TrackOrigins)
- OriginBase =
- getOriginPtrForVAArgument(A->getType(), IRB, OverflowOffset);
OverflowOffset += alignTo(ArgSize, 8);
}
// Take fixed arguments into account for GpOffset and FpOffset,
// but don't actually store shadows for them.
- // TODO(glider): don't call get*PtrForVAArgument() for them.
if (IsFixed)
continue;
if (!ShadowBase)
continue;
- Value *Shadow = MSV.getShadow(A);
- IRB.CreateAlignedStore(Shadow, ShadowBase, kShadowTLSAlignment);
- if (MS.TrackOrigins) {
- Value *Origin = MSV.getOrigin(A);
- unsigned StoreSize = DL.getTypeStoreSize(Shadow->getType());
- MSV.paintOrigin(IRB, Origin, OriginBase, StoreSize,
- std::max(kShadowTLSAlignment, kMinOriginAlignment));
- }
+ IRB.CreateAlignedStore(MSV.getShadow(A), ShadowBase,
+ kShadowTLSAlignment);
}
}
Constant *OverflowSize =
@@ -3413,18 +3380,7 @@ struct VarArgAMD64Helper : public VarArg
Value *Base = IRB.CreatePointerCast(MS.VAArgTLS, MS.IntptrTy);
Base = IRB.CreateAdd(Base, ConstantInt::get(MS.IntptrTy, ArgOffset));
return IRB.CreateIntToPtr(Base, PointerType::get(MSV.getShadowTy(Ty), 0),
- "_msarg_va_s");
- }
-
- /// Compute the origin address for a given va_arg.
- Value *getOriginPtrForVAArgument(Type *Ty, IRBuilder<> &IRB, int ArgOffset) {
- Value *Base = IRB.CreatePointerCast(MS.VAArgOriginTLS, MS.IntptrTy);
- // getOriginPtrForVAArgument() is always called after
- // getShadowPtrForVAArgument(), so __msan_va_arg_origin_tls can never
- // overflow.
- Base = IRB.CreateAdd(Base, ConstantInt::get(MS.IntptrTy, ArgOffset));
- return IRB.CreateIntToPtr(Base, PointerType::get(MSV.getShadowTy(Ty), 0),
- "_msarg_va_o");
+ "_msarg");
}
void unpoisonVAListTagForInst(IntrinsicInst &I) {
@@ -3469,10 +3425,6 @@ struct VarArgAMD64Helper : public VarArg
VAArgOverflowSize);
VAArgTLSCopy = IRB.CreateAlloca(Type::getInt8Ty(*MS.C), CopySize);
IRB.CreateMemCpy(VAArgTLSCopy, 8, MS.VAArgTLS, 8, CopySize);
- if (MS.TrackOrigins) {
- VAArgTLSOriginCopy = IRB.CreateAlloca(Type::getInt8Ty(*MS.C), CopySize);
- IRB.CreateMemCpy(VAArgTLSOriginCopy, 8, MS.VAArgOriginTLS, 8, CopySize);
- }
}
// Instrument va_start.
@@ -3494,9 +3446,6 @@ struct VarArgAMD64Helper : public VarArg
Alignment, /*isStore*/ true);
IRB.CreateMemCpy(RegSaveAreaShadowPtr, Alignment, VAArgTLSCopy, Alignment,
AMD64FpEndOffset);
- if (MS.TrackOrigins)
- IRB.CreateMemCpy(RegSaveAreaOriginPtr, Alignment, VAArgTLSOriginCopy,
- Alignment, AMD64FpEndOffset);
Value *OverflowArgAreaPtrPtr = IRB.CreateIntToPtr(
IRB.CreateAdd(IRB.CreatePtrToInt(VAListTag, MS.IntptrTy),
ConstantInt::get(MS.IntptrTy, 8)),
@@ -3510,12 +3459,6 @@ struct VarArgAMD64Helper : public VarArg
AMD64FpEndOffset);
IRB.CreateMemCpy(OverflowArgAreaShadowPtr, Alignment, SrcPtr, Alignment,
VAArgOverflowSize);
- if (MS.TrackOrigins) {
- SrcPtr = IRB.CreateConstGEP1_32(IRB.getInt8Ty(), VAArgTLSOriginCopy,
- AMD64FpEndOffset);
- IRB.CreateMemCpy(OverflowArgAreaOriginPtr, Alignment, SrcPtr, Alignment,
- VAArgOverflowSize);
- }
}
}
};
Removed: llvm/trunk/test/Instrumentation/MemorySanitizer/X86/vararg_call.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Instrumentation/MemorySanitizer/X86/vararg_call.ll?rev=341540&view=auto
==============================================================================
--- llvm/trunk/test/Instrumentation/MemorySanitizer/X86/vararg_call.ll (original)
+++ llvm/trunk/test/Instrumentation/MemorySanitizer/X86/vararg_call.ll (removed)
@@ -1,110 +0,0 @@
-; RUN: opt < %s -msan -msan-check-access-address=0 -S | FileCheck %s
-; RUN: opt < %s -msan -msan-check-access-address=0 -msan-track-origins=1 -S | FileCheck %s --check-prefixes=CHECK,CHECK-ORIGIN
-; RUN: opt < %s -msan -msan-check-access-address=0 -msan-track-origins=2 -S | FileCheck %s --check-prefixes=CHECK,CHECK-ORIGIN
-
-; Test that shadow and origin are stored for variadic function params.
-
-target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
-target triple = "x86_64-unknown-linux-gnu"
-
-%struct.__va_list_tag = type { i32, i32, i8*, i8* }
-
-define dso_local i32 @test(i32 %a, i32 %b, i32 %c) local_unnamed_addr {
-entry:
- %call = tail call i32 (i32, ...) @sum(i32 3, i32 %a, i32 %b, i32 %c)
- ret i32 %call
-}
-
-; CHECK: store i32 0, {{.*}} @__msan_param_tls {{.*}} i64 8
-; CHECK: store i32 0, {{.*}} @__msan_param_tls {{.*}} i64 16
-; CHECK: store i32 0, {{.*}} @__msan_param_tls {{.*}} i64 24
-; CHECK: store i32 0, {{.*}} @__msan_va_arg_tls {{.*}} i64 8
-; CHECK-ORIGIN: store i32 0, {{.*}} @__msan_va_arg_origin_tls {{.*}} i64 8
-; CHECK: store i32 0, {{.*}} @__msan_va_arg_tls {{.*}} i64 16
-; CHECK-ORIGIN: store i32 0, {{.*}} @__msan_va_arg_origin_tls {{.*}} i64 16
-; CHECK: store i32 0, {{.*}} @__msan_va_arg_tls {{.*}} i64 24
-; CHECK-ORIGIN: store i32 0, {{.*}} @__msan_va_arg_origin_tls {{.*}} i64 24
-
-define dso_local i32 @sum(i32 %n, ...) local_unnamed_addr #0 {
-entry:
- %args = alloca [1 x %struct.__va_list_tag], align 16
- %0 = bitcast [1 x %struct.__va_list_tag]* %args to i8*
- call void @llvm.lifetime.start.p0i8(i64 24, i8* nonnull %0) #2
- call void @llvm.va_start(i8* nonnull %0)
- %cmp9 = icmp sgt i32 %n, 0
- br i1 %cmp9, label %for.body.lr.ph, label %for.end
-
-; CHECK: [[VA_ARG_TLS:%[_0-9a-z]+]] = bitcast {{.*}} @__msan_va_arg_tls
-; CHECK: call void @llvm.memcpy.{{.*}} [[SHADOW_COPY:%[_0-9a-z]+]], {{.*}} [[VA_ARG_TLS]]
-; CHECK-ORIGIN: [[VA_ARG_ORIGIN_TLS:%[_0-9a-z]+]] = bitcast {{.*}} @__msan_va_arg_origin_tls
-; CHECK-ORIGIN: call void @llvm.memcpy{{.*}} [[ORIGIN_COPY:%[_0-9a-z]+]], {{.*}} [[VA_ARG_ORIGIN_TLS]]
-
-; CHECK: call void @llvm.va_start
-; CHECK: call void @llvm.memcpy.{{.*}}, {{.*}} [[SHADOW_COPY]], i{{.*}} [[REGSAVE:[0-9]+]]
-; CHECK-ORIGIN: call void @llvm.memcpy.{{.*}}, {{.*}} [[ORIGIN_COPY]], i{{.*}} [[REGSAVE]]
-
-; CHECK: [[OVERFLOW_SHADOW:%[_0-9a-z]+]] = getelementptr i8, i8* [[SHADOW_COPY]], i{{.*}} [[REGSAVE]]
-; CHECK: call void @llvm.memcpy.{{.*}}[[OVERFLOW_SHADOW]]
-; CHECK-ORIGIN: [[OVERFLOW_ORIGIN:%[_0-9a-z]+]] = getelementptr i8, i8* [[ORIGIN_COPY]], i{{.*}} [[REGSAVE]]
-; CHECK-ORIGIN: call void @llvm.memcpy.{{.*}}[[OVERFLOW_ORIGIN]]
-
-for.body.lr.ph: ; preds = %entry
- %gp_offset_p = getelementptr inbounds [1 x %struct.__va_list_tag], [1 x %struct.__va_list_tag]* %args, i64 0, i64 0, i32 0
- %1 = getelementptr inbounds [1 x %struct.__va_list_tag], [1 x %struct.__va_list_tag]* %args, i64 0, i64 0, i32 3
- %overflow_arg_area_p = getelementptr inbounds [1 x %struct.__va_list_tag], [1 x %struct.__va_list_tag]* %args, i64 0, i64 0, i32 2
- %gp_offset.pre = load i32, i32* %gp_offset_p, align 16
- br label %for.body
-
-for.body: ; preds = %vaarg.end, %for.body.lr.ph
- %gp_offset = phi i32 [ %gp_offset.pre, %for.body.lr.ph ], [ %gp_offset12, %vaarg.end ]
- %sum.011 = phi i32 [ 0, %for.body.lr.ph ], [ %add, %vaarg.end ]
- %i.010 = phi i32 [ 0, %for.body.lr.ph ], [ %inc, %vaarg.end ]
- %fits_in_gp = icmp ult i32 %gp_offset, 41
- br i1 %fits_in_gp, label %vaarg.in_reg, label %vaarg.in_mem
-
-vaarg.in_reg: ; preds = %for.body
- %reg_save_area = load i8*, i8** %1, align 16
- %2 = sext i32 %gp_offset to i64
- %3 = getelementptr i8, i8* %reg_save_area, i64 %2
- %4 = add i32 %gp_offset, 8
- store i32 %4, i32* %gp_offset_p, align 16
- br label %vaarg.end
-
-vaarg.in_mem: ; preds = %for.body
- %overflow_arg_area = load i8*, i8** %overflow_arg_area_p, align 8
- %overflow_arg_area.next = getelementptr i8, i8* %overflow_arg_area, i64 8
- store i8* %overflow_arg_area.next, i8** %overflow_arg_area_p, align 8
- br label %vaarg.end
-
-vaarg.end: ; preds = %vaarg.in_mem, %vaarg.in_reg
- %gp_offset12 = phi i32 [ %4, %vaarg.in_reg ], [ %gp_offset, %vaarg.in_mem ]
- %vaarg.addr.in = phi i8* [ %3, %vaarg.in_reg ], [ %overflow_arg_area, %vaarg.in_mem ]
- %vaarg.addr = bitcast i8* %vaarg.addr.in to i32*
- %5 = load i32, i32* %vaarg.addr, align 4
- %add = add nsw i32 %5, %sum.011
- %inc = add nuw nsw i32 %i.010, 1
- %exitcond = icmp eq i32 %inc, %n
- br i1 %exitcond, label %for.end, label %for.body
-
-for.end: ; preds = %vaarg.end, %entry
- %sum.0.lcssa = phi i32 [ 0, %entry ], [ %add, %vaarg.end ]
- call void @llvm.va_end(i8* nonnull %0)
- call void @llvm.lifetime.end.p0i8(i64 24, i8* nonnull %0) #2
- ret i32 %sum.0.lcssa
-}
-
-
-; Function Attrs: argmemonly nounwind
-declare void @llvm.lifetime.start.p0i8(i64, i8* nocapture) #1
-
-; Function Attrs: nounwind
-declare void @llvm.va_start(i8*) #2
-
-; Function Attrs: nounwind
-declare void @llvm.va_end(i8*) #2
-
-; Function Attrs: argmemonly nounwind
-declare void @llvm.lifetime.end.p0i8(i64, i8* nocapture) #1
-
-
-;declare dso_local i32 @sum(i32, ...) local_unnamed_addr
More information about the llvm-commits
mailing list