Simplifying the emitter handling of 4-byte encoded SSE instructions (#…

…21528)
dotnet · Dec 14, 2018 · 813bd6e · 813bd6e
1 parent bbfafe7
commit 813bd6e
Show file tree

Hide file tree

Showing 5 changed files with 71 additions and 108 deletions.
diff --git a/src/jit/compiler.cpp b/src/jit/compiler.cpp
@@ -2489,14 +2489,6 @@ void Compiler::compSetProcessor()
             codeGen->getEmitter()->SetContainsAVX(false);
             codeGen->getEmitter()->SetContains256bitAVX(false);
         }
-        else if (compSupports(InstructionSet_SSSE3) || compSupports(InstructionSet_AES) ||
-                 compSupports(InstructionSet_PCLMULQDQ))
-        {
-            // Emitter::UseSSE4 controls whether we support the 4-byte encoding for certain
-            // instructions. We need to check if either is supported independently, since
-            // it is currently possible to enable/disable them separately.
-            codeGen->getEmitter()->SetUseSSE4(true);
-        }
     }
 #endif
 #if defined(_TARGET_ARM64_)

diff --git a/src/jit/emit.h b/src/jit/emit.h
@@ -427,7 +427,6 @@ class emitter
 #endif // DEBUG
 
 #ifdef _TARGET_XARCH_
-        SetUseSSE4(false);
         SetUseVEXEncoding(false);
 #endif // _TARGET_XARCH_
     }