Andres Freund <andres(at)anarazel(dot)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