[71iq]

Require a sufficiently wide destination type for pointer-to-integer reinterpret_cast conversions in GNU C++ mode. Microsoft and C compatibility behavior is preserved.

Related discussion: #298
Previously accepted by Mike Herrick as issue EDGcpfe/28892.

Adds rejection tests for object and function pointers and a valid pointer-width conversion test. GCC 13 rejects the negative cases and accepts the positive case.

EDG build, regression testing, and test recordings are pending because the Docker development environment is unavailable. This PR is a draft until those checks are completed.

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

[edgcpp-bot]

AcknowlEDG review results updated for 389fa8f1f379:

• Test - Changes (new)

CI other.

[daveedvdv-nvidia]

See suggested changes.

[daveedvdv-nvidia]

I'd add a small example here.

[daveedvdv-nvidia]

I tracked down why that gpp_mode is there. It's for this test case:

int main()
{
        int *ptr;
        int x;

        x = (int)ptr;
        x = (short)ptr;  // warning in gcc 3.4.6

        return 0;
}

Prior to GCC 4, this was accepted with a warning (https://godbolt.org/z/fG4YcKKr9).
Note that your test case was not accepted even then.
Rather than emulating this 100% accurately, I'd just replace the gpp_mode by gpp_version_is(<40000). That way any pre-GCC-4 code out there that relies on the leniency will continue to work (which should not be a whole lot), and modern code will track GCC 4+.