public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/89148] [AVR] Merge plugin to place C++ vtables in flash memory
[not found] <bug-89148-4@http.gcc.gnu.org/bugzilla/>
@ 2020-04-02 9:14 ` gjl at gcc dot gnu.org
0 siblings, 0 replies; only message in thread
From: gjl at gcc dot gnu.org @ 2020-04-02 9:14 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89148
--- Comment #2 from Georg-Johann Lay <gjl at gcc dot gnu.org> ---
Some remarks:
1. There are AVR devices that don't support named address spaces. You will run
into ICEs with this approach. You'll have to disable it for respective avr
families.
2. The patch sets non-generic address-spaces for objects / types used in the
C++ front-end, and that front-end does not support named address-spaces. How
is it ensured that all transformations that take place in the C++ front-end
handle ASes correctly / consistently?
3. In the case there are problems, the user should be able to switch the
feature off by a command-line option. This option requires documentation, in
particular it's not an optimization option because it changes the ABI.
4. The plugin must be built / installed. So I'd expect some extensions to the
build system like t-avr?
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2020-04-02 9:14 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <bug-89148-4@http.gcc.gnu.org/bugzilla/>
2020-04-02 9:14 ` [Bug target/89148] [AVR] Merge plugin to place C++ vtables in flash memory gjl at gcc dot gnu.org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).