[Autobackout][FunctionalRegression]Revert of change: 156bd92: Enable encoding more imm16 cases directly to BFN.

Ignore constant pool for immediates that can fit into imm16, so they can be
    encoded into instruction directly.

Author: sys-igc

IGC/Compiler/CISACodeGen/PatternMatchPass.cpp

@@ -4619,29 +4619,19 @@ namespace IGC
         pattern->sources[1] = GetSource(s1, false, false, IsSourceOfSample(&I));
         pattern->sources[2] = GetSource(s2, false, false, IsSourceOfSample(&I));
 
-        // BFN can use imm16 in src0 and src2 (not src1).
-        // Directly pass such immediates (or when lossless conversion is possible);
+        // BFN can use imm16 in src0 and src2, check for those;
         // otherwise try to add to constant pool even int32.
-        if (ConstantInt *CI = dyn_cast<ConstantInt>(s0))
+        if (dyn_cast<ConstantInt>(s0) && !s0->getType()->isIntegerTy(16) )
         {
-            if (!s0->getType()->isIntegerTy(16) && !CI->getValue().isIntN(16))
-            {
-                AddToConstantPool(I.getParent(), s0);
-                pattern->sources[0].fromConstantPool = true;
-            }
+            AddToConstantPool(I.getParent(), s0); pattern->sources[0].fromConstantPool = true;
         }
         if (dyn_cast<ConstantInt>(s1))
         {
-            AddToConstantPool(I.getParent(), s1);
-            pattern->sources[1].fromConstantPool = true;
+            AddToConstantPool(I.getParent(), s1); pattern->sources[1].fromConstantPool = true;
         }
-        if (ConstantInt *CI = dyn_cast<ConstantInt>(s2))
+        if (dyn_cast<ConstantInt>(s2) && !s2->getType()->isIntegerTy(16))
         {
-            if (!s2->getType()->isIntegerTy(16) && !CI->getValue().isIntN(16))
-            {
-                AddToConstantPool(I.getParent(), s2);
-                pattern->sources[2].fromConstantPool = true;
-            }
+            AddToConstantPool(I.getParent(), s2); pattern->sources[2].fromConstantPool = true;
         }
 
         AddPattern(pattern);