[PATCH] D94678: [clang][cli] Parse & generate options necessary for LangOptions defaults manually

Michael Spencer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 14 08:58:19 PST 2021


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

LGTM with the comment.



================
Comment at: clang/include/clang/Driver/Options.td:5210-5213
 def finclude_default_header : Flag<["-"], "finclude-default-header">,
-  HelpText<"Include default header file for OpenCL">,
-  MarshallingInfoFlag<LangOpts<"IncludeDefaultHeader">>;
+  HelpText<"Include default header file for OpenCL">;
 def fdeclare_opencl_builtins : Flag<["-"], "fdeclare-opencl-builtins">,
+  HelpText<"Add OpenCL builtin function declarations (experimental)">;
----------------
You should add a comment here explaining why these can't be marshaled.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94678



More information about the cfe-commits mailing list