asavonic marked 3 inline comments as done. ================ Comment at: include/clang/Basic/OpenCLOptions.h:39 @@ +38,3 @@ + + void set(llvm::StringRef Ext, bool Enable = true) { + assert(!Ext.empty() && "Extension is empty."); ---------------- yaxunl wrote: > It seems Enable should be a local variable. This argument is used when `Ext` is not prefixed by '+' or '-'.
https://reviews.llvm.org/D23712 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits