[dpdk-dev] [PATCH] mk: fix build of assembly files for ARM64

Thomas Monjalon thomas.monjalon at 6wind.com
Fri Jan 27 17:20:01 CET 2017


2017-01-18 12:31, Zbigniew Bodek:
> 
> On 17.01.2017 23:45, Thomas Monjalon wrote:
> > 2017-01-17 15:35, zbigniew.bodek at caviumnetworks.com:
> >> -# for now, we don't use as but nasm.
> >> -# AS      = $(CROSS)as
> >
> > It looks to be a very very old comment.
> >
> >> +ifeq ($(CONFIG_RTE_ARCH_X86),y)
> >>  AS        = nasm
> >
> > There is no asm file in DPDK as far as I know.
> > So why would we have a dependency on nasm for x86?
> 
> Yes, there is no such file now. We were to add assembly code but 
> eventually we decided to put it in a separate library. Nevertheless this 
> fix should be harmless if there is no ASM file and will behave correctly 
> if someone will add ASM file in the future.

Please remove the ifeq and keep only $(CROSS)as as below.

> >> +else
> >> +AS        = $(CROSS)as
> >> +endif
> >




More information about the dev mailing list