Fixing regression of "break" logic of MDFramework for Vec.Buff/Vec[]

Adjusting change introduced in: c5a9f76

The problem with original change was that it not only affected warning printing, but the "break" logic also.
So if ShaderDumpEnable was disabled, then it never "broke" and in cases where Vec.Buff is huge e.g 100k entries then
compiling such kernel took hours

Author: bokrzesi

IGC/common/MDFrameWork.cpp

@@ -184,8 +184,10 @@ MDNode* CreateNode(const std::vector<val> &vec, Module* module, StringRef name)
     for (auto it = vec.begin(); it != vec.end(); ++it)
     {
         nodes.push_back(CreateNode(*(it), module, name.str() + "Vec[" + std::to_string(i++) + "]"));
-        if (IGC_IS_FLAG_ENABLED(ShaderDumpEnable) && IGC_IS_FLAG_DISABLED(ShowFullVectorsInShaderDumps) && i > MAX_VECTOR_SIZE_TO_PRINT_IN_SHADER_DUMPS)
+        if (IGC_IS_FLAG_DISABLED(ShowFullVectorsInShaderDumps) && i > MAX_VECTOR_SIZE_TO_PRINT_IN_SHADER_DUMPS)
         {
+          if (IGC_IS_FLAG_ENABLED(ShaderDumpEnable))
+          {
             std::string flagName = "ShowFullVectorsInShaderDumps";
             #ifndef _WIN32
                 flagName = "IGC_" + flagName;
@@ -203,7 +205,8 @@ MDNode* CreateNode(const std::vector<val> &vec, Module* module, StringRef name)
                 printWarningFirstTime = false;
             }
             nodes.push_back(CreateNode(false, module, warningMessage + " " + flagName + " currently equals"));
-            break;
+          }
+          break;
         }
     }
     MDNode* node = MDNode::get(module->getContext(), nodes);