[Lldb-commits] [PATCH] D28804: Provide a substitute to load command of gdb

2017-01-19 Thread Hafiz Abid Qadeer via Phabricator via lldb-commits
abidh closed this revision. abidh added a comment. Committed in 292499. Removed this bit "set PC to its entry address." before committing as it is not being done now. https://reviews.llvm.org/D28804 ___ lldb-commits mailing list

[Lldb-commits] [PATCH] D28804: Provide a substitute to load command of gdb

2017-01-19 Thread Greg Clayton via Phabricator via lldb-commits
clayborg accepted this revision. clayborg added a comment. This revision is now accepted and ready to land. Very nice! https://reviews.llvm.org/D28804 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

[Lldb-commits] [PATCH] D28804: Provide a substitute to load command of gdb

2017-01-19 Thread Hafiz Abid Qadeer via Phabricator via lldb-commits
abidh updated this revision to Diff 84979. abidh added a comment. Changed the name of the option. Added LoadInMemory function to Module and ObjectFile class and moved the functionality of writing to memory there. Removed part which set the pc. https://reviews.llvm.org/D28804 Files:

[Lldb-commits] [PATCH] D28804: Provide a substitute to load command of gdb

2017-01-17 Thread Greg Clayton via Phabricator via lldb-commits
clayborg requested changes to this revision. clayborg added a comment. This revision now requires changes to proceed. We need to agree on options names and move the object file loading code into ObjectFile.cpp. See inlined comments. Comment at:

[Lldb-commits] [PATCH] D28804: Provide a substitute to load command of gdb

2017-01-17 Thread Hafiz Abid Qadeer via Phabricator via lldb-commits
abidh created this revision. For bare-metal targets, lldb was missing a command like 'load' in gdb which can be used to create executable image on the target. This was discussed in http://lists.llvm.org/pipermail/lldb-dev/2016-December/011752.html This diff adds an option to "target module