yaxunl added inline comments. ================ Comment at: lib/Frontend/CompilerInvocation.cpp:1587-1589 @@ -1579,3 +1586,4 @@ + .Case("cl", LangStandard::lang_opencl) .Case("CL1.1", LangStandard::lang_opencl11) .Case("CL1.2", LangStandard::lang_opencl12) .Case("CL2.0", LangStandard::lang_opencl20) ---------------- rsmith wrote: > How about changing these to the lowercase form too, and treating the > uppercase versions as (deprecated) synonyms? (And likewise changing > LangStandards.def to list the lowercase versions, perhaps with the uppercase > versions as aliases.) -cl-std=CL1.1|CL1.2|CL2.0 is defined by OpenCL spec. Allowing lower case may cause some confusion.
-cl-std=cl is not part of OpenCL spec. How about keeping all -cl-std= options big letters and all -std= options small letters? http://reviews.llvm.org/D20630 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits