jdenny added inline comments.

================
Comment at: clang/include/clang/Driver/Options.td:2383
+defm openmp_extensions: BoolFOption<"openmp-extensions",
+  LangOpts<"OpenMPExtensions">, DefaultFalse,
+  PosFlag<SetTrue, [CC1Option, NoArgumentUnused],
----------------
ABataev wrote:
> jdenny wrote:
> > ABataev wrote:
> > > Why do you want to disable it by default?
> > I thought that's what we agreed upon.
> > 
> > I'm personally happy if, in general, all features are enabled by default 
> > and users opt out by requesting strict conformance, but Clang doesn't do 
> > that today even for features from newer OpenMP standards.  Should it do so 
> > for extensions, which appear in no standard?
> > 
> > What would you prefer?
> I would enable it by default, as it may help users in some cases unless there 
> are no other strong opinions.
Well, that promotes my extensions, so I've made that change.


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

https://reviews.llvm.org/D106509

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

Reply via email to