mib added inline comments.

================
Comment at: lldb/scripts/generate-project.py:23
+    with open(header_path, "w") as f:
+        file_contents = (
+            f"#ifndef _OBJ{index}_H\n"
----------------
nit: Is it necessary to put it in a variable or could you have a multiline 
string in the `write` call ?


================
Comment at: lldb/scripts/generate-project.py:183-190
+        f.write("OBJDIR=objs\n")
+        f.write("SDK=$(shell xcrun --show-sdk-path)\n")
+        f.write("SWIFT_FE=swift -frontend\n")
+        f.write("SWIFT_FEFLAGS=-g -Onone -serialize-debugging-options \\\n")
+        f.write("  -sdk $(SDK) -enable-anonymous-context-mangled-names\n")
+        f.write("SWIFTC=swiftc\n")
+        f.write("\n")
----------------
What about here ?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D152569/new/

https://reviews.llvm.org/D152569

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to