我正在使用微控制器进行一些ADC测量。我尝试使用-O2优化来编译以下代码时遇到问题，当代码中存在PrintVal（）函数时，MCU冻结。我进行了一些调试，结果发现当我添加-fno-inline编译器标志时，即使使用PrintVal（）函数，代码也可以正常运行。
这里有一些背景：
AdcIsr.c包含在ADC完成其工作时执行的中断。该文件还包含ISRInit（）函数，该函数初始化将在转换后保存值的变量。在主循环中将等待中断，然后才访问AdcMeas.value。
AdcIsr.c
static volatile uin16_t* isrVarPtr = NULL;
ISR()
{
uint8_t tmp = readAdc();
*isrVarPtr = tmp;
}
void ISRInit(volatile uint16_t *var)
{
isrVarPtr = var;
}
AdcMeas.c
typedef struct{
uint8_t id;
volatile uint16_t value;
}AdcMeas_t;
static AdcMeas_t AdcMeas = {0};
const AdcMeasGetStructPtr()
{
return &AdcMeas;
}
main.c
void PrintVal(const AdcMeas_t* data)
{
printf("AdcMeas %d value: %d\r\n", data->id, data->value);
}
void StartMeasurement()
{
...
AdcOn();
...
}
int main()
{
ISRInit(AdcMeasGetStructPtr()->value);
while(1)
{
StartMeasurement();
WaitForISR();
PrintVal(AdcMeasGetStructPtr());
DelayMs(1000);
}
}
问题：
使用const AdcMeas_t *数据作为PrintVal（）函数的参数是否有问题？我了解AdcMeas.value可能会在中断内更改，并且PrintVal（）可能已过时。
AdcMeas包含一个“通用获取器”。使用这种功能允许对静态结构的只读访问是一种好习惯吗？还是应该实现AdcMeasGetId（）和AdcMeasGetValue函数（请注意，此结构只有2个成员，如果有8个成员该怎么办）？
我知道这段代码有点笨（在while循环中等待中断，这只是一个例子。
一些错误：
您没有头文件，没有库包含文件或您自己的文件。这意味着everything毫无希望地被破坏，直到您修复它为止。如果没有头文件，则无法在C中执行多个文件项目。
*isrVarPtr = tmp;在这里，您写入变量时不受竞争条件的影响。如果主程序分几个步骤读取此变量，则可能会有获取正确数据的风险。您需要防范竞争条件或保证原子访问。
const AdcMeasGetStructPtr()简直是胡言乱语，它里面的return &AdcMeas;不可能用合格的C编译器进行编译。
如果您使用的是旧的但合格的C90编译器，则返回类型将被视为int。否则，如果您使用的是现代C编译器，则即使函数定义也不会编译。因此，似乎您的编译器出了点问题，这比此错误要引起更大的关注。
在C文件中声明typedef struct，然后返回指向它的指针没有任何意义。您需要重新设计该模块。如果只有一个实例（单例），则可以使用getter函数将实例返回到私有结构。但是，如上所述，它需要处理竞争条件。
风格方面的问题：
空括号()几乎总是错误的。这是过时的样式，表示“接受任何参数”。 C ++在这里有所不同。
int main()在微控制器系统中根本没有任何意义。您应该使用一些适合于独立程序的实现定义形式。最常用的格式是void main (void)。
DelayMs(1000);在任何嵌入式系统中都是高度可疑的代码。永远没有理由要挂断MCU一秒钟无用的情况，而这会消耗最大电流。
总体看来，您将从“连续转换” ADC中受益。支持连续转换的ADC会将其最新读取的数据转储到数据寄存器中，您可以在需要时通过轮询将其读取。捕获所有ADC中断实际上仅适用于硬实时系统，信号处理等。