Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

So what's changed between now and when Travis reported this passing for me? See https://travis-ci.org/Linaro/odp/builds/217908015 On Wed, Apr 12, 2017 at 1:58 PM, Brian Brooks wrote: > On Fri, Mar 31, 2017 at 10:02 AM, Maxim Uvarov > wrote: >>

Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

On Fri, Mar 31, 2017 at 10:02 AM, Maxim Uvarov wrote: > rechecked this patch and it fails: > https://travis-ci.org/muvarov/odp/jobs/217195437 > > Maxim. > > On 02/08/17 04:04, Bill Fischofer wrote: >> I've done a fair amount of experimentation with using variants of >>

Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

rechecked this patch and it fails: https://travis-ci.org/muvarov/odp/jobs/217195437 Maxim. On 02/08/17 04:04, Bill Fischofer wrote: > I've done a fair amount of experimentation with using variants of > Brian's suggested macros. This works for C but for C++ it results in > an error message saying

Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

I've done a fair amount of experimentation with using variants of Brian's suggested macros. This works for C but for C++ it results in an error message saying that it can't do relocation and to please compile with -fPIC, which we've already determined is an unacceptable requirement for ODP

Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

Given these issues with obsolete compiler levels, it may be simpler to go with Brian's suggestion and just use some simple macros. I'll post a v4 along those lines. On Tue, Feb 7, 2017 at 7:51 AM, Maxim Uvarov wrote: > On 02/07/17 16:39, Bill Fischofer wrote: >> The

Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

On 02/07/17 16:39, Bill Fischofer wrote: > The errors being reported here are on unchanged code paths with an > older clang version (3.4.0 vs.4.2.1). Not sure why you'd see that here > and not in the unchanged code. > > Also, this run is reporting doxygen issues with the traffic manager > that I

Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

The errors being reported here are on unchanged code paths with an older clang version (3.4.0 vs.4.2.1). Not sure why you'd see that here and not in the unchanged code. Also, this run is reporting doxygen issues with the traffic manager that I don't see running locally: $ make doxygen-doc

Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

I pushed it to github and test failed: https://api.travis-ci.org/jobs/199201799/log.txt?deansi=true Maxim. On 02/07/17 05:51, Brian Brooks wrote: > On 02/06 13:14:17, Bill Fischofer wrote: >> Ping. Brian can you please verify that this now works on your system. >> It works on my Ubuntu 16.04

Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

On 02/06 13:14:17, Bill Fischofer wrote: > Ping. Brian can you please verify that this now works on your system. > It works on my Ubuntu 16.04 system. Just built on: Ubuntu 16.10 - x86_64 Ubuntu 16.04 - aarch64 Arch - aarch64 & x86_64 so the previous build error I reported seems to have

Re: [lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

Ping. Brian can you please verify that this now works on your system. It works on my Ubuntu 16.04 system. A review would also be nice. :) On Thu, Feb 2, 2017 at 4:51 PM, Bill Fischofer wrote: > The ODP_STATIC_ASSERT() macro expands to _Static_assert(), however when >

[lng-odp] [PATCHv4] linux-generic: debug: enable helper use from c++ programs

The ODP_STATIC_ASSERT() macro expands to _Static_assert(), however when used in C++ programs this needs to expand to static_assert(). This resolves Bug https://bugs.linaro.org/show_bug.cgi?id=2852 Reported-by: Moshe Tubul Signed-off-by: Bill Fischofer