From: address@hidden address@hidden on behalf of Arun Sharma address@hidden
Sent: Monday, September 15, 2014 9:51 PM
To: Zhigang Liu
Subject: RE: [Libunwind-devel] Add TileGx platform support
Its looking good. I'll try to merge this weekend.
Arun
Thanks!
--ZhiGang @ Tilera
On Sep 15, 2014 8:32 PM, "Zhigang Liu" < address@hidden> wrote:
Arun,
Did you have time to CR the part for Tilegx platform? All tests are passed for Tilegx so far. Thanks
--ZhiGang @ Tilera
-----Original Message-----
From: libunwind-devel-bounces+zliu=address@hidden [mailto:libunwind-devel-bounces+zliu=address@hidden]
On Behalf Of Zhigang Liu
Sent: Monday, September 08, 2014 4:44 PM
To: Arun Sharma
Cc: address@hidden
Subject: Re: [Libunwind-devel] Add TileGx platform support
Arun,
Tabs in the C source code were removed. And the "docs" hack in the Makefile.am was removed too.
Thanks for CR.
---ZhiGang @ Tilera
https://github.com/zliu2014/libunwind-tilegx
address@hidden:zliu2014/libunwind-tilegx.git
$make check
...
make check-TESTS
make[1]: Entering directory `/u/zliu/WS1/libunwind-orig/latest/libunwind/tests'
PASS: test-proc-info
PASS: test-static-link
PASS: test-strerror
PASS: Gtest-bt
PASS: Ltest-bt
PASS: Gtest-exc
PASS: Ltest-exc
PASS: Gtest-init
PASS: Ltest-init
PASS: Gtest-concurrent
PASS: Ltest-concurrent
PASS: Gtest-resume-sig
PASS: Ltest-resume-sig
PASS: Gtest-resume-sig-rt
PASS: Ltest-resume-sig-rt
FAILURE: expected 13, not 1 frames below signal frame
XFAIL: Gtest-dyn1
FAILURE: expected 13, not 1 frames below signal frame
XFAIL: Ltest-dyn1
PASS: Gtest-trace
PASS: Ltest-trace
PASS: test-async-sig
PASS: test-flush-cache
PASS: test-init-remote
PASS: test-mem
PASS: Ltest-varargs
PASS: Ltest-nomalloc
PASS: Ltest-nocalloc
PASS: Lrs-race
PASS: test-ptrace
PASS: test-setjmp
PASS: run-check-namespace
PASS: run-ptrace-mapper
PASS: run-ptrace-misc
PASS: run-coredump-unwind
PASS: run-coredump-unwind-mdi
======================================================
All 34 tests behaved as expected (2 expected failures) ======================================================
-----Original Message-----
From: address@hidden [mailto:address@hidden] On Behalf Of Arun Sharma
Sent: Sunday, September 07, 2014 5:17 AM
To: Zhigang Liu
Cc: address@hidden
Subject: Re: [Libunwind-devel] Add TileGx platform support
On Fri, Sep 5, 2014 at 7:08 PM, Zhigang Liu <address@hidden> wrote:
> This is the URL of the git repo. at the github. Can you give a code review?
>
> https://github.com/zliu2014/libunwind-tilegx
>
Minor nits:
$ git show | grep -P '\t'
shows some lines with tabs were introduced. Could you convert them to spaces?
+if !ARCH_TILEGX
+SUBDIRS += doc
+endif
Don't think this should be needed. If you don't have all the tools installed, you can work around the problem via:
touch doc/*.man
I haven't reviewed the tilegx specific code carefully. Nothing jumped out at me so far. Happy to pull as long as tests are passing.
-Arun
_______________________________________________
Libunwind-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/libunwind-devel
|