[libc-commits] [libc] [libc] newheadergen: created sys folder for yaml files, make appropriate updates to CMake file for sys folder (PR #98693)

Michael Jones via libc-commits libc-commits at lists.llvm.org
Fri Jul 12 15:23:55 PDT 2024


================
@@ -232,6 +232,15 @@ def main(yaml_file, h_def_file, output_dir, add_function=None):
     output_file_name = Path(h_def_file).stem
     output_file_path = Path(output_dir) / output_file_name
 
+    # with open(yaml_file, "r") as f:
----------------
michaelrj-google wrote:

in that case I'd say it's probably best to leave this out of this patch. Commented out code is often confusing when reading a file

https://github.com/llvm/llvm-project/pull/98693


More information about the libc-commits mailing list