mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-10-22 12:33:33 +02:00
c3a419fcf3
This is r334704 (which was reverted in r334732) with a fix for types like x86_fp80. We need to use getTypeAllocSizeInBits and not getTypeStoreSizeInBits to avoid dropping debug info for such types. Original commit msg: > Summary: > Do not convert a DbgDeclare to DbgValue if the store > instruction only refer to a fragment of the variable > described by the DbgDeclare. > > Problem was seen when for example having an alloca for an > array or struct, and there were stores to individual elements. > In the past we inserted a DbgValue intrinsics for each store, > just as if the store wrote the whole variable. > > When handling store instructions we insert a DbgValue that > indicates that the variable is "undefined", as we do not know > which part of the variable that is updated by the store. > > When ConvertDebugDeclareToDebugValue is used with a load/phi > instruction we assert that the referenced value is large enough > to cover the whole variable. Afaict this should be true for all > scenarios where those methods are used on trunk. If the assert > blows in the future I guess we could simply skip to insert a > dbg.value instruction. > > In the future I think we should examine which part of the variable > that is accessed, and add a DbgValue instrinsic with an appropriate > DW_OP_LLVM_fragment expression. > > Reviewers: dblaikie, aprantl, rnk > > Reviewed By: aprantl > > Subscribers: JDevlieghere, llvm-commits > > Tags: #debug-info > > Differential Revision: https://reviews.llvm.org/D48024 llvm-svn: 334830 |
||
|---|---|---|
| .. | ||
| AArch64 | ||
| AMDGPU | ||
| ARM | ||
| COFF | ||
| Generic | ||
| Inputs | ||
| Lanai | ||
| Mips | ||
| MIR | ||
| MSP430 | ||
| NVPTX | ||
| PDB | ||
| PowerPC | ||
| Sparc | ||
| SystemZ | ||
| WebAssembly | ||
| X86 | ||
| arm-relocs.test | ||
| check-debugify-preserves-analyses.ll | ||
| cross-cu-scope.ll | ||
| debugify-each.ll | ||
| debugify.ll | ||
| debuglineinfo-macho.test | ||
| debuglineinfo-path.ll | ||
| debuglineinfo.test | ||
| debugmacinfo.test | ||
| dwarfdump-64-bit-dwarf.test | ||
| dwarfdump-accel.test | ||
| dwarfdump-debug-frame-simple.test | ||
| dwarfdump-decompression-corrupt.test | ||
| dwarfdump-decompression-error.test | ||
| dwarfdump-dump-flags.test | ||
| dwarfdump-dump-gdbindex.test | ||
| dwarfdump-dwp.test | ||
| dwarfdump-implicit-const.test | ||
| dwarfdump-invalid-line-table.test | ||
| dwarfdump-invalid.test | ||
| dwarfdump-macho-relocs.test | ||
| dwarfdump-macho-universal.test | ||
| dwarfdump-objc.test | ||
| dwarfdump-pubnames.test | ||
| dwarfdump-ranges.test | ||
| dwarfdump-type-units.test | ||
| dwarfdump-zlib.test | ||
| dwo.ll | ||
| invalid-relocations.test | ||
| llvm-symbolizer-split-dwarf-empty.test | ||
| llvm-symbolizer-split-dwarf-no-skel-address.test | ||
| llvm-symbolizer-zlib.test | ||
| llvm-symbolizer.test | ||
| macro_link.ll | ||
| member-pointers.o | ||
| missing-abstract-variable.o | ||
| omit-empty.ll | ||
| pr34186.ll | ||
| pr34672.ll | ||
| precomp.test | ||
| skeletoncu.ll | ||
| strip-DIGlobalVariable.ll | ||
| strip-loop-metadata.ll | ||
| strip-module-flags.ll | ||
| typeunit-header.test | ||
| unrolled-loop-remainder.ll | ||