karnajitw created this revision.
Herald added a subscriber: emaste.
Expression evaluation which takes the non IRInterpreter::Interpret path fails
on i386-*-freebsd. Following changes are done to enable it.
1. Enable i386 ABI creation for freebsd
2. Added an extra argument in ABISysV_i386::Prepar
karnajitw added a comment.
Adding the corresponing reference discussions with freebsd folks.
mmap - https://lists.freebsd.org/pipermail/freebsd-hackers/2017-May/051072.html
x87 tag-
https://lists.freebsd.org/pipermail/freebsd-hackers/2017-June/051166.html
Repository:
rL LLVM
https://reviews
karnajitw added a comment.
Looks like the right thing to do. I will make the changes accordingly.
Repository:
rL LLVM
https://reviews.llvm.org/D34776
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/
karnajitw updated this revision to Diff 104984.
karnajitw added a comment.
Done the changes. Please verify. Tried with RefArray but looks like they don't
work well with copy.
https://reviews.llvm.org/D34776
Files:
include/lldb/Target/Platform.h
source/Plugins/ABI/SysV-i386/ABISysV_i386.cpp
karnajitw marked an inline comment as done.
karnajitw added a comment.
With my test setup on i386-*-freebsd-11.0 (freebsd patched).. There seems to be
quite a good improvement in the number of test failures.
[BEFORE lldb patch]
===
Test Result Summary
===
Test M
karnajitw created this revision.
Herald added a subscriber: emaste.
This is the freebsd equivalent of https://reviews.llvm.org/rL238549.
The fix serves 2 purpose
1. LLDB should handle inferior process signals SIGSEGV/SIGILL/SIGBUS/SIGFPE the
way it is suppose to be handled. Prior to this fix th
karnajitw added a comment.
Adding the associated bugzilla link for this issue
https://bugs.llvm.org/show_bug.cgi?id=23699
[Before Fix]
[/home/karnajitw/llvm_50_public/build.symbol] $ bin/lldb
~/lldb_test_execs/sigsegv
(lldb) target create "/home/karnajitw/lldb_test_execs/sigsegv"
Current executa
karnajitw updated this revision to Diff 106423.
karnajitw added a comment.
Updated the patch. While referring to the linux implementation, looks like I
have added a reinterpret_cast on message.GetFaultAddress which is not required
in the freebsd scenario.
Repository:
rL LLVM
https://reviews
karnajitw updated this revision to Diff 110438.
karnajitw added a comment.
Herald added a subscriber: krytarowski.
Updated the diff to include the changes that were required in the test part.
Please verify.
Repository:
rL LLVM
https://reviews.llvm.org/D35223
Files:
packages/Python/lldbsu
karnajitw added a comment.
Thanks for the info. I will make sure of it in future.
Repository:
rL LLVM
https://reviews.llvm.org/D35223
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-co
10 matches
Mail list logo