- 21 12月, 2017 1 次提交
-
-
由 Koichi Shiraishi 提交于
As of Go 1.8, allows empty GOPATH environment variable.
-
- 19 12月, 2017 3 次提交
-
-
由 aarzilli 提交于
Sometimes delve on Appveyor can't remove the built binary before exiting and gets an "Access is denied" error when trying. See: https://ci.appveyor.com/project/derekparker/delve/build/1527
-
由 aarzilli 提交于
The last entry of the debug_line table is supposed to be valid for every PC address greater than its address.
-
由 aarzilli 提交于
So far we have evaluated the locspec "+0" the same way we evaluate all "+n" locspecs, this means that we turn the current PC into a file:line pair, then we turn back the file:line into a PC address. Normally this is harmless, however all autogenerated code returns the source position "<autogenerated>:1" which resolves back to the very first autogenerated instruction in the code. This messes up the behaviour of the "disassemble" command which uses the locspec "+0" to figure out what code to disassemble if no arguments are passed. We should make +0 always resolve to the current PC (of the given scope) so that clients can use +0 as a default locspec.
-
- 14 12月, 2017 6 次提交
- 08 12月, 2017 4 次提交
-
-
由 aarzilli 提交于
We need to reset the current file and line number. Fixes #1008
-
由 Denis Shevchenko 提交于
-
由 aarzilli 提交于
It can happen if the interface variable escaped, it's out of scope and we autodereference it.
-
由 aarzilli 提交于
On macOS, externally linked programs will have an abbrev for DW_TAG_subprogram without the haschildren flag set. We should handle this case instead of expecting all DW_TAG_subprogram entries to have list of children. Fixes #1034
-
- 29 11月, 2017 4 次提交
-
-
由 aarzilli 提交于
Prints the frame and frame pointer offsets for each frame.
-
由 aarzilli 提交于
When creating a stack trace we should switch between the goroutine stack and the system stack (where cgo code is executed) as appropriate to reconstruct the logical stacktrace. Goroutines that are currently executing on the system stack will have the SystemStack flag set, frames of the goroutine stack will have a negative FrameOffset (like always) and frames of the system stack will have a positive FrameOffset (which is actually just the CFA value for the frame). Updates #935
-
由 aarzilli 提交于
Pass CGO_FLAGS='-O0 -g' to go build to disable optimizations when calling the C compiler.
-
由 Martin Tournoij 提交于
* Support --output for debug, trace, and test commands With the `--output` parameter you can configure the output binary. For example: dlv debug --output /tmp/xxx Will build the binary to `/tmp/xxx`, instead of always putting it as `debug` in the current directory. This ensures that the command always works (even if there is already a file or directory named `debug`) and doesn't write to the source directory. Especially for things like Delve/Vim integration this is a good thing to have, I think. * Address PR feedback and add a test - We don't need to use `filepath.IsAbs()` on startup; I added that because it previously did `"./" + debugname` everywhere, but I don't think that's needed at all, since `pathname` without a leading `./` implies the current directory. - Repurpose the existing `TestIssue398` to also test the `--output` flag. Also fix an issue where tests wouldn't work if `GOPATH` has multiple entries (e..g `GOPATH=$HOME/go:$HOME/mygocode`). - Print an error if we can't remove the debug binary on exit instead of failing silently. Not strictly related to this PR, but a good change to add I think. * Also warn when delve can't remove the binary in test/trace I only added that to debug, but good to issue this warning consistently.
-
- 22 11月, 2017 3 次提交
-
-
由 aarzilli 提交于
These are emitted by C compilers but also by the current development version of the go compiler with the dwarflocationlists flag.
-
由 aarzilli 提交于
-
由 aarzilli 提交于
Either the CPU or the kernel may not support the calls we do when retrieving floating point registers, this isn't an error we should propagate. Also improve the error reporint of pkg/proc/native.fpRegisters. Fixes #1022
-
- 21 11月, 2017 5 次提交
-
-
由 aarzilli 提交于
StepBreakpoints are set on CALL instructions, when they are hit we disassemble the current instruction, figure out the destination address and set a breakpoint after the prologue of the called function. In order to disassemble the current instruction we disassemble the area of memory starting from PC and going to PC+15 (because 15 bytes is the maximum length of one instruction on AMD64). This means that we won't just disassemble one instruction but also a few instructions following it ending with one truncated instruction. This usually works fine but sometimes the disassembler will panic with an array out of bounds error when trying to disassemble a truncated instruction. To avoid this problem this commit changes the funciton disassemble to take one extra parameter, singleInstr, when singleInstr is set disassemble will quit after disassembling a single instruction.
-
由 aarzilli 提交于
If 'iv' is an interface variable with a struct as a concrete value let 'iv.A' evaluate to the access to field 'A' of the concrete value of 'iv'.
-
由 aarzilli 提交于
Replace the unsafe.Pointer type of the buf field of channels with the appropriate array type, allow expressions accessing member field of the channel struct. Fixes #962
-
由 aarzilli 提交于
Conditional breakpoints with unmet conditions would cause next and step to skip the line. This breakpoint changes the Kind field of proc.Breakpoint from a single value to a bit field, each breakpoint object can represent simultaneously a user breakpoint and one internal breakpoint (of which we have several different kinds). The breakpoint condition for internal breakpoints is stored in the new internalCond field of proc.Breakpoint so that it will not conflict with user specified conditions. The breakpoint setting code is changed to allow overlapping one internal breakpoint on a user breakpoint, or a user breakpoint on an existing internal breakpoint. All other combinations are rejected. The breakpoint clearing code is changed to clear the UserBreakpoint bit and only remove the phisical breakpoint if no other bits are set in the Kind field. ClearInternalBreakpoints does the same thing but clearing all bits that aren't the UserBreakpoint bit. Fixes #844
-
由 aarzilli 提交于
Move some duplicate code, related to breakpoints, that was in both backends into a single place. This is in preparation to solve issue #844 (conditional breakpoints make step and next fail) which will make this common breakpoint code more complicated.
-
- 18 11月, 2017 2 次提交
-
-
由 aarzilli 提交于
Since it's just a scratchpad to calculate the new set of registers it makes more sense to have it as a local variable in Next and advanceRegs.
-
由 aarzilli 提交于
Instead of only tracking a few cherrypicked registers in stack.go track all DWARF registers. This is needed for cgo code and for the locationlists emitted by go in 1.10: * The debug_frame sections emitted by C compilers can not be used without tracking all registers * the loclists emitted by go1.10 need all registers of a frame to be interpreted.
-
- 16 11月, 2017 2 次提交
-
-
由 aarzilli 提交于
A thread could terminate between the point when we stop for a breakpoint and the point where we send a stop signal to all threads, if this happens setCurrentBreakpoints will fail with an error. We should tolerate this. For some reason this happens very frequently when running delve on processes with the race detector enabed.
-
由 aarzilli 提交于
Adds test command line flag to compile target fixtures using the -race flag. Multiple tests will fail because of https://github.com/golang/go/issues/22600 but eventually this should work.
-
- 04 11月, 2017 6 次提交
-
-
由 aarzilli 提交于
Some linkers will sometimes insert a zero length entry between the last FDE of a CIE and the next CIE.
-
由 aarzilli 提交于
-
由 aarzilli 提交于
1. Use a slice instead of a map to access standard and extended opcodes (reduces BenchmarkStateMachine from ~12ms/op to ~7ms/op) 2. Cache StateMachine values for the entry point of functions.
-
由 aarzilli 提交于
gosymtab and gopclntab only contain informations about go code, linked C code isn't there, we should use debug_line instead to also cover C. Updates #935
-
由 Davor Kapsa 提交于
-
由 Davor Kapsa 提交于
-
- 25 10月, 2017 1 次提交
-
-
由 aarzilli 提交于
* string to []rune * string to []byte * []rune to string * []byte to string * any pointer to uintptr The string, []rune, []byte conversion pairs aligns this to the go language. The pointer -> uintptr conversion pair is symmetric to the uintptr -> pointer that we already have. Also lets the user specify any size for byte array types instead of just the ones already used by the program, this can be used to read arbitrary memory. Fixes #548, #867
-
- 21 10月, 2017 1 次提交
-
-
由 aarzilli 提交于
-
- 17 10月, 2017 1 次提交
-
-
由 aarzilli 提交于
-
- 26 9月, 2017 1 次提交
-
-
由 aarzilli 提交于
Make 'next' skip deferred functions unless they are called via a panic. Call to a deferred function through 'return' are predictable, if the user wants to step into them 'step' can be used but without this change there is no way to avoid stepping into them. Implements #956
-