libunwind-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Libunwind-devel] Add TileGx platform support


From: Arun Sharma
Subject: Re: [Libunwind-devel] Add TileGx platform support
Date: Sun, 7 Sep 2014 14:47:29 +0530

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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]