Tricky макросъемки в C

Question

У меня такое заявление для PowerPC и не знаю, как это работает

#define XOR(A,D) (*(volatile unsigned int*)(volatile void*)((unsigned char*)0UL + (A)) ^= (D)) 

, который называется (например)

unsigned int a = 3; 
unsigned int b = 50; 
XOR((unsigned int)&a,b); 

С НКУ для стандартный компьютер, который он не компилирует, в моей среде это было, поэтому я спрашиваю себя: wtf ...

Answer 1

Это, по-видимому, излишне запутанный макрос для переключения битов по жестко закодированным адресам. Эти адреса, вероятно, являются адресами регистра памяти. A и D являются ключевыми словами: A = адрес, D = данные. Так обычно, если у вас есть регистр по адресу 0x100, и вы хотите, чтобы переключить его младший значащий бит, вы могли бы написать что-то вроде:

XOR(0x100, 0x01); 

Намного проще версия будет просто:

#define XOR(A, D) ((*(volatile int *)(A)) ^= (D)) 

Это не ясно, почему оригинальный автор использовал несколько приведений и добавил жестко кодированный адрес указателю NULL. По предположению, первый может быть там, чтобы избавиться от предупреждения компилятора, последний, возможно, должен был в какой-то момент разрешить другой регистр.

Answer 2

Это невероятно запутанных способ написания

*(volatile uint32_t*)a ^= b; 

(я выбрал uint32_t, потому что это было указано, что процессор был PowerPC)

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

Не используйте функциональные макросы в этом случае.

Answer 3

Дополнительное преобразование - это уменьшение предупреждений MISRA. Here является ссылкой.