ABataev added inline comments.

================
Comment at: clang/include/clang/Driver/Options.td:1643-1644
   HelpText<"Emit OpenMP code only for SIMD-based constructs.">;
+def fopenmp_enable_irbuilder : Flag<["-"], "fopenmp-enable-irbuilder">, 
Group<f_Group>, Flags<[CC1Option, NoArgumentUnused, HelpHidden]>,
+  HelpText<"Use the experimental OpenMP-IR-Builder codegen path.">;
 def fno_openmp_simd : Flag<["-"], "fno-openmp-simd">, Group<f_Group>, 
Flags<[CC1Option, NoArgumentUnused]>;
----------------
jdoerfert wrote:
> ABataev wrote:
> > Do we need to expose this option to driver or it is enough to have just a 
> > frontend option? If still need to have a driver option, add a test for 
> > driver.
> How do I write a frontend option? Anything that we can query in the 
> lib/CodeGen is fine with me. (I don't even need an option if we turn this on 
> by default to get test coverage right away).
You nedd to move to CC1Options.td file. It means that you can't use it direcly 
when invoke the drive, instead you will need to use `-Xclang -fopenmp-...`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69922



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

Reply via email to