Reverts commit 82fab012eaeb37ea80efb0390232c69e5c7f0109.

"Remove alwaysinline attribute for function calls with aggregate and GAS pointer arguments"

Change-Id: I4e6e3416d6d8d21a3240122f68d972d443678649

Author: dlei6g

IGC/AdaptorCommon/ProcessFuncAttributes.cpp

@@ -354,17 +354,23 @@ bool ProcessFuncAttributes::runOnModule(Module& M)
                 for (auto &arg : F->args())
                 {
                     // If argument contains an opaque type e.g. image, then always inline it.
-                    if (containsOpaque(arg.getType()))
+                    // If argument is a pointer to GAS, always inline it for perf reason.
+                    //
+                    // Note that this workaround should be removed.
+                    if (containsOpaque(arg.getType()) || isSupportedAggregateArgument(&arg) ||
+                        isGASPointer(&arg))
                     {
                         keepAlwaysInline = true;
                         break;
                     }
                 }
 
-                // always inline spirv builtins
-                if (F->getName().startswith(spv::kLLVMName::builtinPrefix))
-                {
-                    keepAlwaysInline = true;
+                // SPIR-V image functions don't contain opaque types for images,
+                // they use i64 values instead.
+                // We need to detect them based on function name.
+                if (F->getName().startswith(spv::kLLVMName::builtinPrefix) &&
+                    F->getName().contains("Image")) {
+                  keepAlwaysInline = true;
                 }
             }