[PATCH] D149019: [Clang] Accept and forward `-fconvergent-functions` in the driver

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 23 10:27:08 PDT 2023


MaskRay added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:972
+  LangOpts<"ConvergentFunctions">, DefaultFalse,
+  NegFlag<SetFalse, [CC1Option], "Assume all functions may be convergent.">,
+  PosFlag<SetTrue, [CC1Option]>>;
----------------
Only one of the positive and negative options need to be a cc1 option.


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5703
                     options::OPT_fno_unique_basic_block_section_names);
+  Args.addOptInFlag(CmdArgs, options::OPT_fconvergent_functions,
+                    options::OPT_fno_convergent_functions);
----------------
Only pass the positive option as cc1 option.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149019



More information about the cfe-commits mailing list