6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2024-07-21 02:29:22 +00:00
Code Issues Projects Releases Wiki Activity

Revert r215415 which causse MSan to crash on a great deal of C++ code.

Browse Source 
I've followed up on the original commit as well.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@215532 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chandler Carruth 2014-08-13 09:19:39 +00:00
parent 918801162a
commit 5e5aa9438d
2 changed files with 0 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/Transforms/Instrumentation/MemorySanitizer.cpp
View File

@ -2356,12 +2356,6 @@ struct MemorySanitizerVisitor : public InstVisitor<MemorySanitizerVisitor> {
VAHelper->visitCallSite(CS, IRB);
}
// If this is a musttail call site, we can't insert propagation code here.
// The return type of the caller must match the callee, so the shadow should
// already be set up for an immediate return.
if (CS.isMustTailCall())
return;
// Now, get the shadow for the RetVal.
if (!I.getType()->isSized()) return;
IRBuilder<> IRBBefore(&I);
@ -2395,10 +2389,6 @@ struct MemorySanitizerVisitor : public InstVisitor<MemorySanitizerVisitor> {
}
void visitReturnInst(ReturnInst &I) {
// Don't propagate shadow between musttail calls and the return.
if (I.getParent()->getTerminatingMustTailCall())
return;
IRBuilder<> IRB(&I);
Value *RetVal = I.getReturnValue();
if (!RetVal) return;

13
test/Instrumentation/MemorySanitizer/msan_basic.ll
View File

@ -878,16 +878,3 @@ define void @MismatchedReturnTypeTailCall(i32 %a) sanitize_memory {
; CHECK-LABEL: define void @MismatchedReturnTypeTailCall
; CHECK: tail call i32 @InnerTailCall
; CHECK: ret void
declare i32 @InnerMustTailCall(i32 %a)
define i32 @MustTailCall(i32 %a) {
%b = musttail call i32 @InnerMustTailCall(i32 %a)
ret i32 %b
}
; Test that 'musttail' is preserved. The ABI should make this work.
; CHECK-LABEL: define i32 @MustTailCall
; CHECK: musttail call i32 @InnerMustTailCall
; CHECK-NEXT: ret i32