Make sure that builtin with noinline (if any) is not affected

by this change at all. This is meant for user functions

Change-Id: Idd366a68c9dd69c6823a1f691a950259a75bc11c

Author: jgu222

IGC/Compiler/CISACodeGen/EstimateFunctionSize.cpp

@@ -309,7 +309,8 @@ void EstimateFunctionSize::checkSubroutine() {
   {
       for (Function& F : *M)
       {
-          if (F.hasFnAttribute(llvm::Attribute::NoInline)) {
+          if (F.hasFnAttribute(llvm::Attribute::NoInline) &&
+              !F.hasFnAttribute(llvm::Attribute::Builtin)) {
               EnableSubroutine = true;
               break;
           }

IGC/common/igc_flags.def

@@ -266,7 +266,7 @@ DECLARE_IGC_REGKEY(bool, ForceDisableSrc0Alpha,        false, "Force the compile
 DECLARE_IGC_REGKEY(bool, EnableLTO,                     true,  "Enable link time optimization")
 DECLARE_IGC_REGKEY(bool, EnableLTODebug,                false, "Enable debug information for LTO")
 DECLARE_IGC_REGKEY(DWORD, FunctionControl,              0,     "Control function inlining/subroutine/stackcall. See value defs in igc_flags.hpp.")
-DECLARE_IGC_REGKEY(bool, EnableOCLNoInlineAttr,         false, "If set, OCL kernel with noinline attribute is honored. Thus, subroutine call is enabled.")
+DECLARE_IGC_REGKEY(bool, EnableOCLNoInlineAttr,         false, "If set, OCL kernel with noinline attribute is honored (only for user function). Thus, subroutine call is enabled.")
 DECLARE_IGC_REGKEY(DWORD, OCLInlineThreshold,           512,   "Setting OCL inline thershold")
 DECLARE_IGC_REGKEY(bool, EnableForceGroupSize,          false, "Enable forcing thread Group Size ForceGroupSizeX and ForceGroupSizeY")
 DECLARE_IGC_REGKEY(DWORD, ForceGroupSizeX,              8, "force group size along X")