Remove threshold for inserting lifetime markers for named temporaries
Now that TailRecursionElimination has been fixed with r222354, the threshold on size for lifetime marker insertion can be removed. This only affects named temporary though, as the patch for unnamed temporaries is still in progress. My previous commit (r222993) was not handling debuginfo correctly, but this could only be seen with some asan tests. Basically, lifetime markers are just instrumentation for the compiler's usage and should not affect debug information; however, the cleanup infrastructure was assuming it contained only destructors, i.e. actual code to be executed, and was setting the breakpoint for the end of the function to the closing '}', and not the return statement, in order to show some destructors have been called when leaving the function. This is wrong when the cleanups are only lifetime markers, and this is now fixed. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@234581 91177308-0d34-0410-b5e6-96231b3b80d8
Showing
- lib/CodeGen/CGCall.cpp 26 additions, 2 deletionslib/CodeGen/CGCall.cpp
- lib/CodeGen/CGCleanup.cpp 20 additions, 1 deletionlib/CodeGen/CGCleanup.cpp
- lib/CodeGen/CGCleanup.h 8 additions, 1 deletionlib/CodeGen/CGCleanup.h
- lib/CodeGen/CGDecl.cpp 33 additions, 26 deletionslib/CodeGen/CGDecl.cpp
- lib/CodeGen/CodeGenFunction.cpp 6 additions, 4 deletionslib/CodeGen/CodeGenFunction.cpp
- lib/CodeGen/CodeGenFunction.h 3 additions, 0 deletionslib/CodeGen/CodeGenFunction.h
- lib/CodeGen/EHScopeStack.h 4 additions, 0 deletionslib/CodeGen/EHScopeStack.h
- test/CodeGen/cleanup-destslot-simple.c 15 additions, 0 deletionstest/CodeGen/cleanup-destslot-simple.c
- test/CodeGen/lifetime-debuginfo-1.c 13 additions, 0 deletionstest/CodeGen/lifetime-debuginfo-1.c
- test/CodeGen/lifetime-debuginfo-2.c 29 additions, 0 deletionstest/CodeGen/lifetime-debuginfo-2.c
- test/CodeGenCXX/destructors.cpp 4 additions, 1 deletiontest/CodeGenCXX/destructors.cpp
- test/CodeGenCXX/nrvo.cpp 13 additions, 2 deletionstest/CodeGenCXX/nrvo.cpp
- test/CodeGenObjC/arc-blocks.m 19 additions, 1 deletiontest/CodeGenObjC/arc-blocks.m
- test/CodeGenObjC/arc-bridged-cast.m 10 additions, 1 deletiontest/CodeGenObjC/arc-bridged-cast.m
- test/CodeGenObjC/arc-literals.m 4 additions, 0 deletionstest/CodeGenObjC/arc-literals.m
- test/CodeGenObjC/arc-precise-lifetime.m 44 additions, 0 deletionstest/CodeGenObjC/arc-precise-lifetime.m
- test/CodeGenObjC/arc-ternary-op.m 12 additions, 0 deletionstest/CodeGenObjC/arc-ternary-op.m
- test/CodeGenObjC/arc.m 98 additions, 11 deletionstest/CodeGenObjC/arc.m
- test/CodeGenObjC/exceptions.m 3 additions, 0 deletionstest/CodeGenObjC/exceptions.m
- test/CodeGenObjCXX/arc-move.mm 10 additions, 0 deletionstest/CodeGenObjCXX/arc-move.mm
Loading
Please register or sign in to comment