Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-06 Thread Honnappa Nagarahalli
On 6 April 2017 at 13:54, Maxim Uvarov wrote: > I'm ok with this patch. That can go to master, not need for api-next. > > Maxim. We need this to be in api-next as well so that we can drop this from V3. Thank you, Honnapppa > > On 04/06/17 20:09, Brian Brooks wrote: >>

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-06 Thread Maxim Uvarov
I'm ok with this patch. That can go to master, not need for api-next. Maxim. On 04/06/17 20:09, Brian Brooks wrote: > On 04/05 01:24:44, Dmitry Eremin-Solenikov wrote: >> On 04.04.2017 23:34, Brian Brooks wrote: >>> On 04/04 23:27:51, Dmitry Eremin-Solenikov wrote: On 04.04.2017 23:26,

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-06 Thread Brian Brooks
On 04/05 01:24:44, Dmitry Eremin-Solenikov wrote: > On 04.04.2017 23:34, Brian Brooks wrote: > > On 04/04 23:27:51, Dmitry Eremin-Solenikov wrote: > >> On 04.04.2017 23:26, Brian Brooks wrote: > >>> On 04/04 23:04:10, Dmitry Eremin-Solenikov wrote: > On 04.04.2017 21:47, Brian Brooks wrote: >

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Dmitry Eremin-Solenikov
On 04.04.2017 23:34, Brian Brooks wrote: > On 04/04 23:27:51, Dmitry Eremin-Solenikov wrote: >> On 04.04.2017 23:26, Brian Brooks wrote: >>> On 04/04 23:04:10, Dmitry Eremin-Solenikov wrote: On 04.04.2017 21:47, Brian Brooks wrote: > Signed-off-by: Brian Brooks

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Dmitry Eremin-Solenikov
On 05.04.2017 00:31, Brian Brooks wrote: > On 04/04 14:53:23, Bill Fischofer wrote: >> On Tue, Apr 4, 2017 at 2:45 PM, Brian Brooks wrote: >>> On 04/04 21:55:06, Maxim Uvarov wrote: this patch should be not part of this serries and can go to master as standalone

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Bill Fischofer
On Tue, Apr 4, 2017 at 4:31 PM, Brian Brooks wrote: > On 04/04 14:53:23, Bill Fischofer wrote: >> On Tue, Apr 4, 2017 at 2:45 PM, Brian Brooks wrote: >> > On 04/04 21:55:06, Maxim Uvarov wrote: >> >> this patch should be not part of this serries and

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Brian Brooks
On 04/04 14:53:23, Bill Fischofer wrote: > On Tue, Apr 4, 2017 at 2:45 PM, Brian Brooks wrote: > > On 04/04 21:55:06, Maxim Uvarov wrote: > >> this patch should be not part of this serries and can go to master as > >> standalone patch. > > > > Can you help to cherry pick

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Brian Brooks
On 04/04 23:27:51, Dmitry Eremin-Solenikov wrote: > On 04.04.2017 23:26, Brian Brooks wrote: > > On 04/04 23:04:10, Dmitry Eremin-Solenikov wrote: > >> On 04.04.2017 21:47, Brian Brooks wrote: > >>> Signed-off-by: Brian Brooks > >> > >> Brian, > >> > >> how does this fail

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Dmitry Eremin-Solenikov
On 04.04.2017 23:26, Brian Brooks wrote: > On 04/04 23:04:10, Dmitry Eremin-Solenikov wrote: >> On 04.04.2017 21:47, Brian Brooks wrote: >>> Signed-off-by: Brian Brooks >> >> Brian, >> >> how does this fail with clang on ARMv8? Could you please include error >> message in

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Brian Brooks
On 04/04 23:04:10, Dmitry Eremin-Solenikov wrote: > On 04.04.2017 21:47, Brian Brooks wrote: > > Signed-off-by: Brian Brooks > > Brian, > > how does this fail with clang on ARMv8? Could you please include error > message in the commit message? Well, you can't pass -mcx16

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Dmitry Eremin-Solenikov
On 04.04.2017 21:47, Brian Brooks wrote: > Signed-off-by: Brian Brooks Brian, how does this fail with clang on ARMv8? Could you please include error message in the commit message? -- With best wishes Dmitry

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Maxim Uvarov
On 04/04/17 22:45, Brian Brooks wrote: > On 04/04 21:55:06, Maxim Uvarov wrote: >> this patch should be not part of this serries and can go to master as >> standalone patch. > > Can you help to cherry pick such patches to master? They need to remain > in this patch series until they are cherry

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Bill Fischofer
On Tue, Apr 4, 2017 at 2:45 PM, Brian Brooks wrote: > On 04/04 21:55:06, Maxim Uvarov wrote: >> this patch should be not part of this serries and can go to master as >> standalone patch. > > Can you help to cherry pick such patches to master? They need to remain > in this

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Brian Brooks
On 04/04 21:55:06, Maxim Uvarov wrote: > this patch should be not part of this serries and can go to master as > standalone patch. Can you help to cherry pick such patches to master? They need to remain in this patch series until they are cherry picked to master and master is sync'd into

Re: [lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Maxim Uvarov
this patch should be not part of this serries and can go to master as standalone patch. No need of camel case is shor description. Long description in git comment is required. Maxim. On 04/04/17 21:47, Brian Brooks wrote: > Signed-off-by: Brian Brooks > --- >

[lng-odp] [API-NEXT PATCH v2 01/16] Fix native Clang build on ARMv8

2017-04-04 Thread Brian Brooks
Signed-off-by: Brian Brooks --- configure.ac | 30 -- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/configure.ac b/configure.ac index 419c136f..da091ab1 100644 --- a/configure.ac +++ b/configure.ac @@ -304,20 +304,22 @@