[libc][docs] Parse inline macro_value from YAML in docgen (#189118)

kaladron · web-flow · commit d6ff5e77781d · 2026-03-27T22:19:29.000Z
The docgen script was previously hardcoded to assume all implemented
macros must be placed in a *-macros.h header. This updates docgen to
read inline macro_value properties directly from the source YAML files,
correctly recognizing them as implemented.
diff --git a/libc/utils/docgen/header.py b/libc/utils/docgen/header.py
@@ -43,13 +43,28 @@ def __init__(self, header_name: str):
         self.docgen_root = Path(__file__).parent
         self.libc_root = self.docgen_root.parent.parent
         self.docgen_yaml = self.docgen_root / Path(header_name).with_suffix(".yaml")
+        # Path to libc/include/<name>.yaml
+        self.include_yaml = Path(
+            self.libc_root, "include", Path(header_name).with_suffix(".yaml")
+        )
         self.fns_dir = Path(self.libc_root, "src", self.stem)
         self.macros_dir = Path(self.libc_root, "include", "llvm-libc-macros")
 
     def macro_file_exists(self) -> bool:
         for _ in self.__get_macro_files():
             return True
 
+        if self.include_yaml.exists():
+            import yaml
+
+            parsed = yaml.safe_load(self.include_yaml.read_text(encoding="utf-8"))
+            if (
+                parsed
+                and "macros" in parsed
+                and any("macro_value" in m for m in parsed["macros"])
+            ):
+                return True
+
         return False
 
     def fns_dir_exists(self) -> bool:
@@ -75,6 +90,15 @@ def implements_macro(self, m_name: str) -> bool:
             if f"#define {m_name}" in f.read_text():
                 return True
 
+        if self.include_yaml.exists():
+            import yaml
+
+            parsed = yaml.safe_load(self.include_yaml.read_text(encoding="utf-8"))
+            if parsed and "macros" in parsed:
+                for macro in parsed["macros"]:
+                    if macro.get("macro_name") == m_name and "macro_value" in macro:
+                        return True
+
         return False
 
     def __get_macro_files(self) -> Generator[Path, None, None]:
