Andres Freund <and...@anarazel.de> writes: > When running cpluspluscheck I get many many complaints like > /tmp/pg-test-repo/src/include/port/atomics/arch-x86.h:143:23: warning: ISO > C++17 does not allow ‘register’ storage class specifier [-Wregister]
Interesting, I don't see that here. > It seems we should just remove the use of register? I have a vague idea that it was once important to say "register" if you are going to use the variable in an asm snippet that requires it to be in a register. That might be wrong, or it might be obsolete even if once true. We could try taking these out and seeing if the buildfarm complains. (If so, maybe -Wno-register would help?) regards, tom lane