Re: [Lldb-commits] [PATCH] D21221: Fix for PrintStackTraces

2016-06-15 Thread Jason Molenda via lldb-commits
jasonmolenda added a comment. Regarding the part of the patch where eh_frame is used unconditionally on Linux systems, I'm hesitant to make a change like that (beyond the notes I mentioned earlier about how I'd do it in FuncUnwinders instead of down in RegisterContextLLDB). Modern gcc's put ou

Re: [Lldb-commits] [PATCH] D21221: Fix for PrintStackTraces

2016-06-15 Thread Ravitheja Addepally via lldb-commits
ravitheja added a comment. @jasonmolenda The approach suggested seems promising, I look forward to it, if u want I can test it at my end ? once I see it. http://reviews.llvm.org/D21221 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http:

[Lldb-commits] [lldb] r272800 - Allow runtimes to execute custom LLVM ModulePasses over the expression IR

2016-06-15 Thread Luke Drummond via lldb-commits
Author: ldrumm Date: Wed Jun 15 11:19:46 2016 New Revision: 272800 URL: http://llvm.org/viewvc/llvm-project?rev=272800&view=rev Log: Allow runtimes to execute custom LLVM ModulePasses over the expression IR During expression evaluation, the ClangExpressionParser preforms a number of hard-coded fi

Re: [Lldb-commits] [PATCH] D17027: [expression evaluator] Allow runtimes to execute custom LLVM ModulePasses over the generated IR at various stages after expression compilation.

2016-06-15 Thread Luke Drummond via lldb-commits
ldrumm abandoned this revision. ldrumm added a comment. committed in http://reviews.llvm.org/rL272800 http://reviews.llvm.org/D17027 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commit

Re: [Lldb-commits] Buildbot numbers for the last week of 6/05/2016 - 6/11/2016

2016-06-15 Thread Galina Kistanova via lldb-commits
Maybe a comma-separated attachments? The tables relay positioning and font. Thanks Galina On Tue, Jun 14, 2016 at 6:59 PM, Sean Silva wrote: > Some of these tables are getting quite mangled somewhere along the line > (email client? mailing list handling? idk). Could you re-send with the > tab

[Lldb-commits] [lldb] r272844 - Set TERM env var for Xcode gtests

2016-06-15 Thread Todd Fiala via lldb-commits
Author: tfiala Date: Wed Jun 15 16:31:02 2016 New Revision: 272844 URL: http://llvm.org/viewvc/llvm-project?rev=272844&view=rev Log: Set TERM env var for Xcode gtests Not sure what changed, but something outside our code is failing one of the EditLine gtests on OS X CI (and locally) before the gt

[Lldb-commits] [PATCH] D21422: Fix typo in eOpenOptionDontFollowSymlinks

2016-06-15 Thread Francis Ricci via lldb-commits
fjricci created this revision. fjricci added reviewers: clayborg, labath. fjricci added a subscriber: lldb-commits. Fix capitalization http://reviews.llvm.org/D21422 Files: include/lldb/Host/File.h source/Host/common/File.cpp source/Target/Platform.cpp Index: source/Target/Platform.cpp ==

Re: [Lldb-commits] [PATCH] D21280: Allow installing watchpoints at less than 8-byte alligned addresses for AArch64 targets

2016-06-15 Thread Muhammad Omair Javaid via lldb-commits
omjavaid updated this revision to Diff 60942. omjavaid added a comment. Herald added a subscriber: srhines. I have added a test cases that tests all possibilities supported by current configuration. Tests pass on Nexus 9 and aarch64-linux-gnu (hikey board). LGTM? http://reviews.llvm.org/D2128

Re: [Lldb-commits] [PATCH] D21280: Allow installing watchpoints at less than 8-byte alligned addresses for AArch64 targets

2016-06-15 Thread Pavel Labath via lldb-commits
labath accepted this revision. labath added a comment. This revision is now accepted and ready to land. Thanks for adding the test. I have some nits about some details of the test. If you agree with them, you can commit the updated version without additional review. Comment at