Skip to content
This repository has been archived by the owner on Dec 1, 2021. It is now read-only.

The flag of FUNC_TIME_MEASUREMENT should be set in compiling all ELF files #924

Closed
tk26eng opened this issue Mar 13, 2020 · 0 comments
Closed

Comments

@tk26eng
Copy link
Contributor

tk26eng commented Mar 13, 2020

Now -DFUNC_TIME_MEASUREMENT is set only for compiling lm_fpga.elf in Makefile.
But I think it's better for us to set the flag to all ELF files in generating them.
Actually time measurement results are very useful in development.
Most developer wants the information in debug.
We get little longer logs from those ELF files if the flag is enabled, but it's no problem because those ELF files are only for debug.

lm_x86: CXX = g++
lm_x86: FLAGS += $(INCLUDES) -O3 -std=c++14 -DUSE_PNG -pthread -g
lm_x86: CXXFLAGS +=
lm_x86_avx: CXX = g++
lm_x86_avx: FLAGS += $(INCLUDES) -O3 -std=c++14 -mavx2 -mfma -DUSE_AVX -DUSE_PNG -pthread -g -fopenmp
lm_x86_avx: CXXFLAGS +=
lm_aarch64: CXX = aarch64-linux-gnu-g++
lm_aarch64: FLAGS += $(INCLUDES) -std=c++14 -O3 -DUSE_NEON -DUSE_PNG -pthread -g -fopenmp
lm_aarch64: CXXFLAGS +=
lm_arm: CXX = arm-linux-gnueabihf-g++
lm_arm: FLAGS += $(INCLUDES) -std=c++14 -O3 -DUSE_NEON -DUSE_PNG -DAARCH32 -mcpu=cortex-a9 -mfpu=neon -mthumb -s -pthread -g -fopenmp
lm_arm: CXXFLAGS +=
lm_fpga: CXX = arm-linux-gnueabihf-g++
lm_fpga: FLAGS += $(INCLUDES) -std=c++14 -O3 -DUSE_NEON -DRUN_ON_FPGA -DUSE_PNG -DAARCH32 -mcpu=cortex-a9 -mfpu=neon -mthumb -pthread -g -fopenmp -DFUNC_TIME_MEASUREMENT
lm_fpga: CXXFLAGS +=

What do you think about this ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant