[PATCH] D66101: gn build: Give cmake sync script an opt-in --write flag

[Peter Collingbourne via Phabricator via llvm-commits]

pcc accepted this revision.
pcc added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/utils/gn/build/sync_source_lists_from_cmake.py:34
+    if tokloc == -1: raise ValueError(gn_file + ': Failed to find source list')
+    if gn_contents.find(srcs_tok, tokloc + 1) != -1:
+        raise ValueError(gn_file + ': Multiple source lists')
----------------
Should this also refuse to operate if it sees a `# NOSORT` comment? I think the only instance of that is in `clang/lib/Headers`, though -- maybe we can remove that comment now that we have a mechanism for automatic updates?


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

https://reviews.llvm.org/D66101