further address review comments

Signed-off-by: chenwei.sun <chenwei.sun@intel.com>

Author: tomflinda

clang/lib/DPCT/MigrateScript/MigrateCmakeScript.cpp

@@ -581,23 +581,18 @@ static const std::vector<std::string> BuiltinMigrationRules = {
 
 static void reserveBuiltinMigrationRules() {
   for (const auto &Rule : BuiltinMigrationRules) {
-    MetaRuleObject::PatternRewriter PrePR;
-    PrePR.BuildScriptSyntax = Rule;
-    PrePR.RuleId = "builtin_rule_" + Rule;
-
-    auto Iter = CmakeBuildInRules.find(PrePR.BuildScriptSyntax);
-    if (Iter != CmakeBuildInRules.end()) {
-      if (PrePR.Priority == RulePriority::Takeover &&
-          Iter->second.Priority > PrePR.Priority) {
-        CmakeBuildInRules[PrePR.BuildScriptSyntax] = PrePR;
-      } else {
-        llvm::outs() << "[Warning]: Two migration rules (Rule:" << PrePR.RuleId
-                     << ", Rule:" << Iter->second.RuleId
-                     << ") are duplicated, the migration rule (Rule:"
-                     << PrePR.RuleId << ") is ignored.\n";
-      }
-    } else {
-      CmakeBuildInRules[PrePR.BuildScriptSyntax] = PrePR;
+    MetaRuleObject::PatternRewriter BuiltinPR;
+    BuiltinPR.BuildScriptSyntax = Rule;
+    BuiltinPR.RuleId = "builtin_rule_" + Rule;
+
+    auto &PR = CmakeBuildInRules[Rule];
+    if (PR.Priority != RulePriority::Takeover) {
+      PR = BuiltinPR;
+    } else if (PR.Priority == RulePriority::Takeover) {
+      llvm::outs() << "[Warning]: Two migration rules (Rule:"
+                   << BuiltinPR.RuleId << ", Rule:" << PR.RuleId
+                   << ") are duplicated, the migration rule (Rule:"
+                   << BuiltinPR.RuleId << ") is ignored.\n";
     }
   }
 }