ekieri marked an inline comment as done.
ekieri added inline comments.

================
Comment at: clang/include/clang/Driver/Options.td:5756
+
+}
+
----------------
awarzynski wrote:
> [nit] IMHO this file lacks comments, hence the suggestion :) 
Sure!

On the topic of this file's structure, I found the nested "let Flags" quite 
confusing -- Setting "Flags" inside "def version" does not work, as it would be 
overwritten by the outer "let Flags" on line 4820. But that is a different 
patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122542

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

Reply via email to