[PATCH] D143430: [C2x] Implement the `unreachable` macro for WG14 N2826

2023-02-15 Thread Aaron Ballman via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGa9797d7d2d78: [C2x] Implement the `unreachable` macro for WG14 N2826 (authored by aaron.ballman). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143430/new/

[PATCH] D143430: [C2x] Implement the `unreachable` macro for WG14 N2826

2023-02-15 Thread Erich Keane via Phabricator via cfe-commits
erichkeane accepted this revision. erichkeane added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Headers/stddef.h:108 +__STDC_VERSION__ >= 202000L +#define unreachable() __builtin_unreachable() +#endif /*

[PATCH] D143430: [C2x] Implement the `unreachable` macro for WG14 N2826

2023-02-15 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added inline comments. Comment at: clang/lib/Headers/stddef.h:108 +__STDC_VERSION__ >= 202000L +#define unreachable() __builtin_unreachable() +#endif /* defined(__need_STDDEF_H_misc) && >= C23 */ erichkeane wrote: > Is this REALLY required to

[PATCH] D143430: [C2x] Implement the `unreachable` macro for WG14 N2826

2023-02-14 Thread Erich Keane via Phabricator via cfe-commits
erichkeane added inline comments. Comment at: clang/lib/Headers/stddef.h:108 +__STDC_VERSION__ >= 202000L +#define unreachable() __builtin_unreachable() +#endif /* defined(__need_STDDEF_H_misc) && >= C23 */ Is this REALLY required to be a macro by standard?

[PATCH] D143430: [C2x] Implement the `unreachable` macro for WG14 N2826

2023-02-06 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman created this revision. aaron.ballman added reviewers: jyknight, clang-language-wg, efriedma. Herald added a project: All. aaron.ballman requested review of this revision. Herald added a project: clang. This exposes `__builtin_unreachable` as the expansion for the `unreachable`