[libcxx-commits] [libcxx] [libc++] [NFC] Documentation: Add `_LIBCPP_PUSH_MACROS` and `_LIBCPP_POP_MACROS` (PR #79963)

Hristo Hristov via libcxx-commits libcxx-commits at lists.llvm.org
Tue Jan 30 00:20:00 PST 2024


https://github.com/H-G-Hristov created https://github.com/llvm/llvm-project/pull/79963

I got tripped twice after: 7b4622514d232ce5f7110dd8b20d90e81127c467 Let's at least mention these in the `Contributing.rst` doc.

>From 02945acd281844f840dd287910a2e62ea6cf8851 Mon Sep 17 00:00:00 2001
From: Hristo Hristov <hghristov.rmm at gmail.com>
Date: Tue, 30 Jan 2024 10:19:34 +0200
Subject: [PATCH] [libc++] [NFC] Documentation: Add `_LIBCPP_PUSH_MACROS` and
 `_LIBCPP_POP_MACROS`

I got tripped twice after: 7b4622514d232ce5f7110dd8b20d90e81127c467 Let's at least mention these in the `Contributing.rst` doc.
---
 libcxx/docs/Contributing.rst | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libcxx/docs/Contributing.rst b/libcxx/docs/Contributing.rst
index 3ff8c15a969b0..596d86ef22449 100644
--- a/libcxx/docs/Contributing.rst
+++ b/libcxx/docs/Contributing.rst
@@ -162,6 +162,7 @@ sure you don't forget anything:
 
 - Did you add the relevant feature test macro(s) for your feature? Did you update the ``generate_feature_test_macro_components.py`` script with it?
 - Did you run the ``libcxx-generate-files`` target and verify its output?
+- If needed, did you add `_LIBCPP_PUSH_MACROS` and `_LIBCPP_POP_MACROS` to the relevant headers?
 
 The review process
 ==================



More information about the libcxx-commits mailing list