[NFC] Improve for options_compile.cpp

1. Check return value for consume_front function
2. Pass by reference to avoid copy StringRef

Author: haonanya1

options_compile.cpp

@@ -259,7 +259,7 @@ std::string EffectiveOptionsFilter::processOptions(const OpenCLArgList &args,
   std::map<std::string, bool> extMap;
   llvm::SmallVector<llvm::StringRef> extVec;
   llvm::SplitString(PCH_EXTENSION, extVec, ",");
-  for(auto ext : extVec)
+  for (auto &ext : extVec)
     extMap.insert({ext.str(), true});
 #else
   std::map<std::string, bool> extMap{
@@ -287,10 +287,11 @@ std::string EffectiveOptionsFilter::processOptions(const OpenCLArgList &args,
 
   auto parseClExt = [&](const std::string &clExtStr) {
     llvm::StringRef clExtRef(clExtStr);
-    clExtRef.consume_front("-cl-ext=");
+    if (!clExtRef.consume_front("-cl-ext="))
+      return;
     llvm::SmallVector<llvm::StringRef, 32> parsedExt;
     clExtRef.split(parsedExt, ',');
-    for (auto ext : parsedExt) {
+    for (auto &ext : parsedExt) {
       char sign = ext.front();
       bool enabled = sign != '-';
       llvm::StringRef extName = ext;