public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "redi at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c++/110619] Dangling pointer returned from constexpr function converts in nullptr
Date: Mon, 07 Aug 2023 09:12:57 +0000	[thread overview]
Message-ID: <bug-110619-4-5uO9ikzC6T@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-110619-4@http.gcc.gnu.org/bugzilla/>

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110619

--- Comment #9 from Jonathan Wakely <redi at gcc dot gnu.org> ---
(In reply to Fedor Chelnokov from comment #8)
> This program must be rejected because of relational comparison of unrelated
> pointers

I disagree. If GCC were to define the behaviour of returning invalid pointers
during constant evaluation as converting them to null pointers, then those
wouldn't be unrelated pointers. They'd be null pointers. And the comparison
would be well defined.

But I don't think that conversion was ever intended as the defined behaviour,
just an unintended consequence. If it was intended, it should have been
documented at
https://gcc.gnu.org/onlinedocs/gcc/C_002b_002b-Implementation.html

The fact that we _don't_ do any such conversion should also be documented
there, of course. We should say that other uses of invalid pointers do not
modify the pointer values, so copying them preserves the invalid value (without
trapping, except on Itanium?) and pointer comparisons and pointer arithmetic
are undefined because they don't point to an object.

      parent reply	other threads:[~2023-08-07  9:13 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-10 18:39 [Bug c++/110619] New: " fchelnokov at gmail dot com
2023-07-10 18:43 ` [Bug c++/110619] " pinskia at gcc dot gnu.org
2023-07-10 18:44 ` pinskia at gcc dot gnu.org
2023-07-10 19:04 ` fchelnokov at gmail dot com
2023-07-12  1:51 ` de34 at live dot cn
2023-07-13 19:02 ` ppalka at gcc dot gnu.org
2023-07-26  1:45 ` cvs-commit at gcc dot gnu.org
2023-08-07  5:04 ` peter at cordes dot ca
2023-08-07  7:41 ` fchelnokov at gmail dot com
2023-08-07  9:12 ` redi at gcc dot gnu.org [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-110619-4-5uO9ikzC6T@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).