I have some small comments to the patch: * You should also update gcc/doc/invoke.texi
Sergey Ostanevich wrote:
index 0026683..84911a0 100644 --- a/gcc/c-family/c.opt +++ b/gcc/c-family/c.opt
...
+Wopenmp-simd +C C++ Var(openmp_simd) Warning +Warn about omp simd construct is overridden by cost model
--- a/gcc/fortran/lang.opt +++ b/gcc/fortran/lang.opt
...
+Wopenmp-simd +Fortran Warning +Warn about omp simd construct is overridden by cost model
As the option files get merged, using Wopenmp-simd Fortran ; Documented in C is sufficient.
+fsimd-cost-model= +Common Joined RejectNegative Enum(vect_cost_model) Var(flag_simd_cost_model) Init(VECT_COST_MODEL_DEFAULT) +Specifies the cost model for vectorization in loops marked with omp simd
I wonder whether we need to care about Cilk Plus' "#pragma simd" in this summary.
+ if (LOOP_VINFO_LOOP (loop_vinfo)->force_vect) + { + warning (OPT_Wopenmp_simd, "Vectorization did not happen for the loop ", + "labeled as simd"); + } +
The warning line is too long. Tobias