Как установить порядок компиляции файлов в Atmel Studio?

Question

[Nickellick]

Допустим, у меня есть файл main.c, в котором я пишу:

#define SSEG_DIG_PORT PORTB
#include "include/led_display.h"

Внутри файла led_display.h написано следующее:

#ifndef SSEG_DIG_PORT
	#define SSEG_DIG_PORT PORTA
	#warning "SSEG_DIG_PORT is not defined! Using PORTA instead"
#endif /* SSEG_DIG_PORT */

В итоге при компиляции я получаю этот warning, и, само собой, устанавливаю не тот порт. Будто бы компилятор сперва отдельно от всего, игнорируя что написано в main.c, отдельно компилирует led_display.c
Что в такой ситуации делать? Только писать свой makefile? Для примера, если использовать те же задержки из util/delay.h, то там таких проблем нет. Т.е. main.c выглядит следующим образом:

#define F_CPU 16000000UL
#include <util/delay.h>

И в таком случае мы не получаем никаких warning'ов, хотя если не определить макрос F_CPU, то компилятор предупредит, что из-за этого у нас могут быть проблемы

P.S. Работаю из Atmel Studio 7

Answer 1

[res2001]

Непосредственно к Atmel Studio это не относится. Это особенность работы С/С++ компиляторов.

Будто бы компилятор сперва отдельно от всего, игнорируя что написано в main.c, отдельно компилирует led_display.c

В корень зрите. Каждый файл исходных кодов в Си и С++ компилируется отдельно от других. Это называется единицей трансляции.
Объединяются объектные файлы только на этапе линковки. В процессе сборки исполняемого файла компиляция исходников и линковка объектных файлов разделены.

Макросы, которые должны быть общими для всех исходников помещайте в отдельные хедеры и включайте эти хедеры в соответствующие файлы исходных кодов.
Как вариант макросы можно задавать глобально опцией компиляции для всех файлов в проекте. Можно через makefile или опциями в проекте в IDE.
В вашем случае можно просто этот макрос поместить в led_display.h.

Comments

[Nickellick]

Любопытно, спасибо! Тогда почему тоже самое не справедливо для delay.h (это из стандартной библиотеки AVR, может поэтому?), которому тоже по-хорошему нужно определить F_CPU? Т.е. на этапе компиляции он умеет "подхватывать" это значение из main.c

[res2001]

Nickellick,

на этапе компиляции он умеет "подхватывать" это значение из main.c

Файлы заголовков, которые подключаются в .c файлы не являются отдельными единицами трансляции. Препроцессор Си сначала создает объединенный файл в который включаются все заголовки из директив #include рекурсивно и непосредственно код из .c файла.
Вот этот объединенный файл уже и будет полной единицей трансляции, которая передается компилятору.
Исходя из вышеописанного если вы объявите макрос до включения заголовка, то этот макрос может быть использован в заголовке при включении.

На самом деле процесс может быть оптимизирован, но проще думать об этом именно так, а детали уже не столь важны.

[res2001]

Включение заголовков из #include в полную единицу трансляции, конечно происходит в том порядке, в котором директивы указаны в коде. Т.е. по факту вместо директивы #include у вас будет содержимое заголовка после препроцессора.

Кстати в компиляторе обычно есть опции, позволяющие сохранять эти промежуточные файлы после обработки препроцесором вашего кода.

[Nickellick]

res2001,

Исходя из вышеописанного если вы объявите макрос до включения заголовка, то этот макрос может быть использован в заголовке при включении.

Но ведь в коде всё именно так и описано, т.е. в main.c сперва я объявляю макрос, а потом уже добавляю заголовочный файл

#define SSEG_DIG_PORT PORTB
#include "include/led_display.h"

И всё равно получаю warning, который описал в led_display.h

[Nickellick]

Я не знаю насколько легально показывать код из стандартной библиотеки avr, но по своей сути в заголовке delay.h также написано приблизительно такой же код, только про другой макрос, который нужно определить заранее

#ifndef SSEG_DIG_PORT
  #define SSEG_DIG_PORT PORTA
  #warning "SSEG_DIG_PORT is not defined! Using PORTA instead"
#endif /* SSEG_DIG_PORT */

Т.е. поведение компилятора для моего led_display.h и заголовочного файла из библиотеки при одинаковом коде разное

[res2001]

Nickellick, Из вашего вопроса следует, что предупреждение появляется при компиляции led_display.c, а не main.c. А сейчас у вас уже другая версия :) Вы определитесь как-то. Причем это вполне может быть, т.к. макрос определен только в main.c.
Почему предупреждение при компиляции led_display.c и как этого избежать я вам объяснил. При компиляции main.c предупреждения быть не должно.

Внимательно смотрите вывод компилятора, там можно увидеть при компиляции какого файла случилось предупреждение.

[res2001]

Nickellick, Можете запостить сюда под спойлер кусок вывода компилятора с предупреждением.

[Nickellick]

res2001
Да, думаю сразу стоит прояснить - компилировал я сразу весь проект (решение), я как я глянул в его Makefile, что он мне сгенерировал - он компилирует и main.c и led_display.c

Возможно действительно не так сформулировал вопрос
В общем мне нужно чтобы я мог взять, определить какие-либо макросы, скажем, в main.c, и чтобы эти определения попали в led_display

Вывод компилятора:

spoiler

Building file: ../src/led_display.c
Invoking: AVR/GNU C Compiler : 5.4.0
"C:\Program Files (x86)\Atmel\Studio\7.0\toolchain\avr8\avr8-gnu-toolchain\bin\avr-gcc.exe" -x c -funsigned-char -funsigned-bitfields -DDEBUG -I"C:\Program Files (x86)\Atmel\Studio\7.0\Packs\atmel\ATmega_DFP\1.2.209\include" -O1 -ffunction-sections -fdata-sections -fpack-struct -fshort-enums -g2 -Wall -Wextra -mmcu=atmega32a -B "C:\Program Files (x86)\Atmel\Studio\7.0\Packs\atmel\ATmega_DFP\1.2.209\gcc\dev\atmega32a" -c -std=gnu99 -MD -MP -MF "src/led_display.d" -MT"src/led_display.d" -MT"src/led_display.o" -o "src/led_display.o" "../src/led_display.c"
In file included from ../src/led_display.c:8:0:
../src/include/led_display.h:18:3: warning: #warning "SSEG_DIG_PORT is not defined! Using PORTA instead" [-Wcpp]
#warning "SSEG_DIG_PORT is not defined! Using PORTA instead"
^
../src/include/led_display.h:23:3: warning: #warning "SSEG_DDR is not defined! Using DDRA instead" [-Wcpp]
#warning "SSEG_DDR is not defined! Using DDRA instead"
^
../src/include/led_display.h:28:2: warning: #warning "SSEG_POV_PORT is not defined! Using PORTC instead" [-Wcpp]
#warning "SSEG_POV_PORT is not defined! Using PORTC instead"
^
../src/include/led_display.h:33:2: warning: #warning "SSEG_POV_DDR is not defined! Using DDRC instead" [-Wcpp]
#warning "SSEG_POV_DDR is not defined! Using DDRC instead"
^
Finished building: ../src/led_display.c
Building file: ../src/main.c
Invoking: AVR/GNU C Compiler : 5.4.0
"C:\Program Files (x86)\Atmel\Studio\7.0\toolchain\avr8\avr8-gnu-toolchain\bin\avr-gcc.exe" -x c -funsigned-char -funsigned-bitfields -DDEBUG -I"C:\Program Files (x86)\Atmel\Studio\7.0\Packs\atmel\ATmega_DFP\1.2.209\include" -O1 -ffunction-sections -fdata-sections -fpack-struct -fshort-enums -g2 -Wall -Wextra -mmcu=atmega32a -B "C:\Program Files (x86)\Atmel\Studio\7.0\Packs\atmel\ATmega_DFP\1.2.209\gcc\dev\atmega32a" -c -std=gnu99 -MD -MP -MF "src/main.d" -MT"src/main.d" -MT"src/main.o" -o "src/main.o" "../src/main.c"
Finished building: ../src/main.c
Building target: FreqCounter.elf
Invoking: AVR/GNU Linker : 5.4.0
"C:\Program Files (x86)\Atmel\Studio\7.0\toolchain\avr8\avr8-gnu-toolchain\bin\avr-gcc.exe" -o FreqCounter.elf src/led_display.o src/main.o -Wl,-Map="FreqCounter.map" -Wl,--start-group -Wl,-lm -Wl,--end-group -Wl,--gc-sections -mmcu=atmega32a -B "C:\Program Files (x86)\Atmel\Studio\7.0\Packs\atmel\ATmega_DFP\1.2.20
9\gcc\dev\atmega32a"
Finished building target: FreqCounter.elf
"C:\Program Files (x86)\Atmel\Studio\7.0\toolchain\avr8\avr8-gnu-toolchain\bin\avr-objcopy.exe" -O ihex -R .eeprom -R .fuse -R .lock -R .signature -R .user_signatures "FreqCounter.elf" "FreqCounter.hex"
"C:\Program Files (x86)\Atmel\Studio\7.0\toolchain\avr8\avr8-gnu-toolchain\bin\avr-objcopy.exe" -j .eeprom --set-section-flags=.eeprom=alloc,load --change-section-lma .eeprom=0 --no-change-warnings -O ihex "FreqCounter.elf" "FreqCounter.eep" || exit 0
"C:\Program Files (x86)\Atmel\Studio\7.0\toolchain\avr8\avr8-gnu-toolchain\bin\avr-objdump.exe" -h -S "FreqCounter.elf" > "FreqCounter.lss"
"C:\Program Files (x86)\Atmel\Studio\7.0\toolchain\avr8\avr8-gnu-toolchain\bin\avr-objcopy.exe" -O srec -R .eeprom -R .fuse -R .lock -R .signature -R .user_signatures "FreqCounter.elf" "FreqCounter.srec"
"C:\Program Files (x86)\Atmel\Studio\7.0\toolchain\avr8\avr8-gnu-toolchain\bin\avr-size.exe" "FreqCounter.elf"
text data bss dec hex filename
536 10 4 550 226 FreqCounter.elf

Прошу прощения, подправил форматирование как смог, но мог что-то не углядеть и что-нибудь могло перенестись на следующую строку. Спасибо!

[res2001]

Nickellick, Все верно. Предупреждение выдается при компиляции led_display.c.
Просто выделите все общие макросы в отдельный заголовочный файл, в этом же файле можете подключать и include/led_display.h. В .c файлах (во всех где нужно), вместо определения макросов просто включаете этот заголовок. Профит.

[Nickellick]

res2001, Понял, спасибо
Просто теперь любопытно почему такой метод срабатывает если я буду работать с delay.h из стандартной библиотеки, при абсолютно таком же коде

[res2001]

Nickellick,

Просто теперь любопытно почему такой метод срабатывает если я буду работать с delay.h из стандартной библиотеки, при абсолютно таком же коде

Вы заблуждаетесь. Проанализируйте внимательно код, изучите вывод компилятора.