User Details
- User Since
- Jun 6 2021, 5:11 PM (236 w, 5 d)
Today
Yesterday
Sun, Dec 14
LGTM
Fri, Dec 12
Wed, Dec 10
Tue, Dec 9
Mon, Dec 8
Sat, Dec 6
Could we yoink the logic from python.mk for this?
Fri, Dec 5
Thu, Dec 4
Patch looks fine. Approved for commit once you feel like you have tested it sufficiently.
There seems to be a stray gcc in the file.
Wed, Dec 3
I recommend that instead of !defined(__cplusplus), maybe we do !defined(unreachable). This should also cover C codebases with their own unreachable macros and matches what we do for offsetof.
Tue, Dec 2
Mon, Dec 1
Sun, Nov 30
Fri, Nov 28
But didn't you propose making exactly that change?
For testing, the gold standard is to build the port and execute the test suite on as many platforms we support as possible, on all currently supported OS versions (that is, 13.5, 14.3, and 15.0). If you don't have access to some architectures, you can skip testing there, but especially for an infrastructure port like this it is a good idea to be thorough.