merged changes from review

Author: GlassBeaver

lib/DXIL/DxilModule.cpp

@@ -1831,10 +1831,10 @@ bool DxilModule::StripNamesSensitiveToDebug() {
 
   if (!GetShaderModel()->IsLib()) {
     // Strip struct names
-    vector<StructType *> structTypes = m_pModule->getIdentifiedStructTypes();
-    unsigned nextStructId = 0;
-    for (StructType *structType : structTypes) {
-      if (!structType->hasName())
+    SmallVector<StructType *,8> structTypes = m_pModule->getIdentifiedStructTypes();
+    unsigned NextStructId = 0;
+    for (StructType *ST : structTypes) {
+      if (!ST->hasName())
         continue;
 
       StringRef Name = structType->getName();

tools/clang/tools/dxcompiler/dxclinker.cpp

@@ -416,9 +416,8 @@ HRESULT STDMETHODCALLTYPE DxcLinker::Link(
             &ShaderHashContent, pReflectionStream, pRootSigStream, nullptr,
             nullptr);
 
-        if (opts.StripDebug) {
+        if (opts.StripDebug)
           inputs.pM->GetOrCreateDxilModule().StripNamesSensitiveToDebug();
-        }
 
         if (needsValidation) {
           valHR = dxcutil::ValidateAndAssembleToContainer(inputs);

tools/clang/tools/dxcompiler/dxcompilerobj.cpp

@@ -1048,9 +1048,8 @@ class DxcCompiler : public IDxcCompiler3,
 
           inputs.pVersionInfo = static_cast<IDxcVersionInfo *>(this);
 
-          if (opts.StripDebug) {
+          if (opts.StripDebug)
             inputs.pM->GetOrCreateDxilModule().StripNamesSensitiveToDebug();
-          }
 
           if (needsValidation) {
             valHR = dxcutil::ValidateAndAssembleToContainer(inputs);