Attention is currently required from: Arthur Heymans, Martin L Roth. Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62175 )
Change subject: xcompile,clang: increase the number of bracket-depth for CPP ......................................................................
Patch Set 1:
(1 comment)
File util/xcompile/xcompile:
https://review.coreboot.org/c/coreboot/+/62175/comment/76cd6b75_6c703f73 PS1, Line 241: 2048
While I get the idea of "let's just fix this so it never happens again", maybe we should change to 512 or so first? I mean even exceeding 256 levels of nested brackets seems like it might be a sign that something should be rewritten. Going straight to 2048 seems excessive.
According to the patch that introduced this option, it has an effect on stack usage, presumably just by changing the option. https://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20130218/074645.htm...
GPIO configuration on Intel platforms seems to use a ton of nested brackets with macros. 512 & 1024 was not enough so I bumped it to 2048.