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

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

Reply via email to