Port in PSTR macro changes from arduino core

[earlephilhower]

esp8266/Arduino#6565

[mikee47]

As the register keyword is deprecated and, in C++17, effectively banned, perhaps it should be removed? Unless you're aware of a specific use for it within the xtensa toolchain?

http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4340

EDIT: Although the keyword still appears in the C standards I suspect it doesn't really do anything any more. But I could be wrong!

https://stackoverflow.com/questions/37456494/why-was-the-register-keyword-created

[earlephilhower]

No, I think you're right. Or, at least, C++17 makes it a warning deprecated and it breaks CI. I remember having to fix this in our Arduino core.

Although I'm not exactly sure what register you're referring to. Can you provide a link?

I'll also make asm->asm or whatever is the "right" way to make that happen, as you commented in the Sming repo IIRC.

[mikee47]

https://gcc.gnu.org/onlinedocs/gcc/Extended-Asm.html

The asm keyword is a GNU extension. When writing code that can be compiled with -ansi and the various -std options, use __asm__ instead of asm (see Alternate Keywords).

[mikee47]

Although I'm not exactly sure what register you're referring to. Can you provide a link?

It's used extensively within Arduino, but there are two instances within pgmspace.h in pgm_read_byte_inlined() and pgm_read_word_inlined():

static inline uint8_t pgm_read_byte_inlined(const void* addr) {
  register uint32_t res;
  pgm_read_with_offset(addr, res);
  return (uint8_t) res;     /* This masks the lower byte from the returned word */
}

[mikee47]

I've just attempted a Linux compile with optimisations disabled, and got this:

In file included from /home/mike/sandboxes/sming-dev/Sming/Arch/Esp8266/Components/libc/include/sys/pgmspace.h:43,
                 from /home/mike/sandboxes/sming-dev/Sming/Wiring/FakePgmSpace.h:14,
                 from /home/mike/sandboxes/sming-dev/Sming/Wiring/FakePgmSpace.cpp:11:
/opt/esp-quick-toolchain/xtensa-lx106-elf/xtensa-lx106-elf/include/sys/pgmspace.h: In function 'uint32_t pgm_read_dword_unaligned(const void*)':
/opt/esp-quick-toolchain/xtensa-lx106-elf/xtensa-lx106-elf/include/sys/pgmspace.h:97:1: error: a15 cannot be used in asm here
   97 | }
      | ^

It's objecting to the A15 clobber, if I remove that it's fine.

(Note: I only ran this test at -O0 to check if libstdc++ got implicated, but thought I should mention it.)

[earlephilhower]

Got it on the format changes, thanks. I guess since I didn't include the right pgmspace.h they didn't pop up for me. :)

For a15 thing is probably a clobber list problem for me. I'll take a look at it. We never use -O0 because code blow up so bad, but I can see how it would make GDB much nicer...

[mikee47]

@earlephilhower Did you forget to change asm -> __asm__ as per #3 (comment) ?

[earlephilhower]

I did try the code with gcc9.2 and std=c++17 and -O0 in arduino with no errors. Are you sure about having an issue with asm(xxxx) as opposed to the 1-line asm ?

[mikee47]

This is the error I get:

/home/mike/sandboxes/esp-quick-toolchain/xtensa-lx106-elf.x86_64/xtensa-lx106-elf/include/sys/pgmspace.h:72:3: error: expected ')' before ':' token
   72 |   pgm_read_with_offset(addr, res);
      |   ^~~~~~~~~~~~~~~~~~~~