This is a clean-up change for fixing EOL whitespace

Change-Id: I91c714f4476bcc2b168596bc3840a94adc337bf6

Author: paigeale

IGC/AdaptorCommon/AddImplicitArgs.cpp

@@ -76,7 +76,7 @@ bool AddImplicitArgs::runOnModule(Module &M)
     m_pMdUtils = getAnalysis<MetaDataUtilsWrapper>().getMetaDataUtils();
     CodeGenContext* ctx = getAnalysis<CodeGenContextWrapper>().getCodeGenContext();
 
-    // Update function signatures 
+    // Update function signatures
     // Create new functions with implicit args
     for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I)
     {
@@ -146,7 +146,7 @@ bool AddImplicitArgs::runOnModule(Module &M)
         // Loop over the argument list, transferring uses of the old arguments over to
         // the new arguments
         updateNewFuncArgs(pFunc, pNewFunc, &implicitArgs);
-        
+
         // Map old func to new func
         funcsMapping[pFunc] = pNewFunc;
 
@@ -168,7 +168,7 @@ bool AddImplicitArgs::runOnModule(Module &M)
     // Function declarations are changing, this needs to be reflected in the metadata.
     MetadataBuilder mbuilder(&M);
     auto &FuncMD = ctx->getModuleMetaData()->FuncMD;
-    for (auto i : funcsMapping) 
+    for (auto i : funcsMapping)
     {
         auto oldFuncIter = m_pMdUtils->findFunctionsInfoItem(i.first);
         m_pMdUtils->setFunctionsInfoItem(i.second, oldFuncIter->second);
@@ -183,7 +183,7 @@ bool AddImplicitArgs::runOnModule(Module &M)
         }
     }
     m_pMdUtils->save(M.getContext());
-    
+
     //Return if any error
     if (!(getAnalysis<CodeGenContextWrapper>().getCodeGenContext()->oclErrorMessage.empty()))
     {
@@ -209,7 +209,7 @@ bool AddImplicitArgs::runOnModule(Module &M)
     return true;
 }
 
-FunctionType* AddImplicitArgs::getNewFuncType(Function* pFunc, const ImplicitArgs* pImplicitArgs) 
+FunctionType* AddImplicitArgs::getNewFuncType(Function* pFunc, const ImplicitArgs* pImplicitArgs)
 {
     // Add all explicit parameters
     FunctionType* pFuncType = pFunc->getFunctionType();
@@ -220,12 +220,12 @@ FunctionType* AddImplicitArgs::getNewFuncType(Function* pFunc, const ImplicitArg
     {
         newParamTypes.push_back((*pImplicitArgs)[i].getLLVMType(pFunc->getContext()));
     }
-    
+
     // Create new function type with explicit and implicit parameter types
     return FunctionType::get( pFunc->getReturnType(),newParamTypes, pFunc->isVarArg());
 }
 
-void AddImplicitArgs::updateNewFuncArgs(llvm::Function* pFunc, llvm::Function* pNewFunc, const ImplicitArgs* pImplicitArgs) 
+void AddImplicitArgs::updateNewFuncArgs(llvm::Function* pFunc, llvm::Function* pNewFunc, const ImplicitArgs* pImplicitArgs)
 {
     // Loop over the argument list, transferring uses of the old arguments over to
     // the new arguments, also transferring over the names as well.
@@ -279,7 +279,7 @@ void AddImplicitArgs::updateNewFuncArgs(llvm::Function* pFunc, llvm::Function* p
         llvm::Value* newArg = &(*currArg);
         if ((*I).getType() != currArg->getType())
         {
-            // fix opague type mismatch on %opencl.image... 
+            // fix opague type mismatch on %opencl.image...
             std::string str0;
             llvm::raw_string_ostream s(str0);
             currArg->getType()->print(s);
@@ -420,7 +420,7 @@ void AddImplicitArgs::replaceAllUsesWithNewOCLBuiltinFunction(CodeGenContext* ct
             llvm::Value* arg = cInst->getOperand(i);
             if (arg->getType() != new_arg_iter->getType())
             {
-                // fix opague type mismatch on %opencl... 
+                // fix opague type mismatch on %opencl...
                 std::string str0;
                 llvm::raw_string_ostream s(str0);
                 arg->getType()->print(s);
@@ -660,7 +660,7 @@ void BuiltinCallGraphAnalysis::traveseCallGraphSCC(const std::vector<CallGraphNo
         if (!f || f->isDeclaration())
             continue;
         // Fail on variadic functions.
-        if (f->isVarArg()) 
+        if (f->isVarArg())
         {
             std::string Msg = "Invalid user defined function being processed: ";
             Msg += f->getName();

IGC/AdaptorCommon/AddImplicitArgs.hpp

@@ -84,7 +84,7 @@ namespace IGC
         ~AddImplicitArgs() {}
 
         /// @brief  Provides name of pass
-        virtual llvm::StringRef getPassName() const override 
+        virtual llvm::StringRef getPassName() const override
         {
             return "AddImplictArgs";
         }
@@ -103,11 +103,11 @@ namespace IGC
         virtual bool runOnModule(llvm::Module &M) override;
 
     private:
-        /// @brief  Create the type of the new function, 
+        /// @brief  Create the type of the new function,
         ///         including all explicit and needed impliict parameters
         /// @param  pFunc           The old function
         /// @param  pImplicitArgs   The impliict arguments needed by this function
-        /// @returns    The new functuin type 
+        /// @returns    The new functuin type
         llvm::FunctionType* getNewFuncType(llvm::Function* pFunc, const ImplicitArgs* pImplicitArgs);
 
         /// @brief  Transfers uses of old arguments to new arguments, sets names of all arguments

IGC/AdaptorCommon/IRUpgrader/IRUpgrader.hpp

@@ -34,7 +34,7 @@ namespace llvm
 }
 namespace IGC
 {
-    /// Transform legacy resource access intrinsics taking an integer representation to 
+    /// Transform legacy resource access intrinsics taking an integer representation to
     /// the new intrinsics taking pointer representation
     llvm::Pass* CreateUpgradeResourceIntrinsic();
     ///Transfor legacy gen intrisics with prefix "@genx." => "@llvm.genx."

IGC/AdaptorCommon/IRUpgrader/UpgraderResourceAccess.cpp

@@ -119,7 +119,7 @@ void UpgradeResourceAccess::ChangeIntrinsic(CallInst& C, GenISAIntrinsic::ID ID)
     case GenISAIntrinsic::GenISA_sampleCptr:
     case GenISAIntrinsic::GenISA_sampleBCptr:
     case GenISAIntrinsic::GenISA_sampleLptr:
-    case GenISAIntrinsic::GenISA_sampleDptr: 
+    case GenISAIntrinsic::GenISA_sampleDptr:
     case GenISAIntrinsic::GenISA_sampleKillPix: {
         types.push_back(C.getType());
         types.push_back(C.getOperand(0)->getType());
@@ -176,7 +176,7 @@ void UpgradeResourceAccess::visitCallInst(CallInst& C)
         return;
     }
 
-    //Note : This upgrader pass is handling legacy intrinsics as well, 
+    //Note : This upgrader pass is handling legacy intrinsics as well,
     // hence we might be dealing with intrinsics starting with @genx.
     if(C.getCalledFunction()->getName().contains("genx.GenISA.sample."))
     {
@@ -206,7 +206,7 @@ void UpgradeResourceAccess::visitCallInst(CallInst& C)
     {
         ChangeIntrinsic(C, GenISAIntrinsic::GenISA_ldmsptr);
     }
-    else if(C.getCalledFunction()->getName().equals("llvm.genx.GenISA.ldmcs") || 
+    else if(C.getCalledFunction()->getName().equals("llvm.genx.GenISA.ldmcs") ||
         C.getCalledFunction()->getName().equals("genx.GenISA.ldmcs"))
     {
         ChangeIntrinsic(C, GenISAIntrinsic::GenISA_ldmcsptr);
@@ -232,7 +232,7 @@ class UpgradeGenIntrinsicPrefix : public ModulePass, public InstVisitor<UpgradeG
 {
 public:
     UpgradeGenIntrinsicPrefix() : ModulePass(ID) {}
-    
+
     static char ID;
     void getAnalysisUsage(llvm::AnalysisUsage &AU) const override
     {
@@ -263,7 +263,7 @@ bool UpgradeGenIntrinsicPrefix::runOnModule(llvm::Module &M)
 }
 
 
-namespace IGC 
+namespace IGC
 {
 Pass* CreateUpgradeResourceIntrinsic()
 {

IGC/AdaptorCommon/ImplicitArgs.cpp

@@ -75,7 +75,7 @@ Type* ImplicitArg::getLLVMType(LLVMContext& context) const
     // the allocation size, but not to determine the LLVM type, because every
     // work-item sees a scalar.
     Type* baseType = nullptr;
-    switch(m_valType) 
+    switch(m_valType)
     {
     case BYTE:
         baseType = Type::getInt8Ty(context);
@@ -116,14 +116,14 @@ Type* ImplicitArg::getLLVMType(LLVMContext& context) const
     }
 }
 
-unsigned int ImplicitArg::getNumberElements() const 
+unsigned int ImplicitArg::getNumberElements() const
 {
     return m_nbElement;
 }
 
 VISA_Type ImplicitArg::getVISAType(const DataLayout& DL) const
 {
-    switch(m_valType) 
+    switch(m_valType)
     {
     case BYTE:
         return VISA_Type::ISA_TYPE_B;
@@ -150,7 +150,7 @@ VISA_Type ImplicitArg::getVISAType(const DataLayout& DL) const
 
     return VISA_Type::ISA_TYPE_UD;
 }
- 
+
 unsigned int ImplicitArg::getPointerSize(const DataLayout& DL) const {
   switch (m_valType) {
   case CONSTPTR:    return DL.getPointerSize(ADDRESS_SPACE_CONSTANT);
@@ -191,11 +191,11 @@ WIAnalysis::WIDependancy ImplicitArg::getDependency() const {
     return m_dependency;
 }
 
-unsigned int ImplicitArg::getAllocateSize(const DataLayout& DL) const 
+unsigned int ImplicitArg::getAllocateSize(const DataLayout& DL) const
 {
     unsigned int elemSize = 0;
 
-    switch(m_valType) 
+    switch(m_valType)
     {
     case BYTE:
         elemSize = 1;
@@ -234,7 +234,7 @@ bool ImplicitArg::isLocalIDs() const {
                  m_argType == ImplicitArg::LOCAL_ID_Z);
 }
 
-ImplicitArgs::ImplicitArgs(const llvm::Function& func , const MetaDataUtils* pMdUtils) 
+ImplicitArgs::ImplicitArgs(const llvm::Function& func , const MetaDataUtils* pMdUtils)
 {
     if (IMPLICIT_ARGS.size() == 0)
     {
@@ -327,14 +327,14 @@ const ImplicitArg& ImplicitArgs::operator[](unsigned int i) const {
 
 unsigned int ImplicitArgs::getArgIndex(ImplicitArg::ArgType argType) {
     assert(this->size() > 0 && "There are no implicit arguments!");
-    
+
     // Find the first appearance of the given implicit arg type
     unsigned int implicitArgIndex = 0;
-    for (implicitArgIndex = 0; implicitArgIndex < this->size(); ++implicitArgIndex) 
+    for (implicitArgIndex = 0; implicitArgIndex < this->size(); ++implicitArgIndex)
     {
         ImplicitArg::ArgType type = getArgType(implicitArgIndex);
-        if (type == argType) 
-        { 
+        if (type == argType)
+        {
             break;
         }
     }
@@ -418,8 +418,8 @@ void ImplicitArgs::addImplicitArgs(llvm::Function& F, SmallVectorImpl<ImplicitAr
     pMdUtils->save(F.getParent()->getContext());
 }
 
-void ImplicitArgs::addImageArgs(llvm::Function& F, ImplicitArg::ArgMap& argMap, MetaDataUtils* pMdUtils) 
-{    
+void ImplicitArgs::addImageArgs(llvm::Function& F, ImplicitArg::ArgMap& argMap, MetaDataUtils* pMdUtils)
+{
     FunctionInfoMetaDataHandle funcInfo = pMdUtils->getFunctionsInfoItem(&F);
     for (int i = 0; i < numImageArgTypes; ++i)
     {
@@ -471,16 +471,16 @@ void ImplicitArgs::addNumberedArgs(llvm::Function& F, ImplicitArg::ArgMap& argMa
   pMdUtils->save(F.getParent()->getContext());
 }
 
-// Add one implicit argument for each pointer argument to global or constant buffer. 
+// Add one implicit argument for each pointer argument to global or constant buffer.
 // Note that F is the original input function (ie, without implicit arguments).
 void ImplicitArgs::addBufferOffsetArgs(llvm::Function& F, IGCMD::MetaDataUtils* pMdUtils, IGC::ModuleMetaData *modMD)
 {
     ImplicitArg::ArgMap OffsetArgs;
     FunctionInfoMetaDataHandle funcInfoMD =
         pMdUtils->getFunctionsInfoItem(const_cast<Function*>(&F));
-    
+
     assert(modMD->FuncMD.find(&F) != modMD->FuncMD.end());
-    
+
     FunctionMetaData* funcMD = &modMD->FuncMD.find(&F)->second;
     for (auto& Arg : F.args() )
     {
@@ -494,7 +494,7 @@ void ImplicitArgs::addBufferOffsetArgs(llvm::Function& F, IGCMD::MetaDataUtils*
         }
 
         int argNo = Arg.getArgNo();
-        
+
         std::string argbaseType = "";
         if (funcMD->m_OpenCLArgBaseTypes.size() > (unsigned)argNo)
             argbaseType = funcMD->m_OpenCLArgBaseTypes[argNo];
@@ -516,17 +516,17 @@ void ImplicitArgs::addBufferOffsetArgs(llvm::Function& F, IGCMD::MetaDataUtils*
     }
 }
 
-unsigned int ImplicitArgs::size() const { 
+unsigned int ImplicitArgs::size() const {
     return m_funcInfoMD->size_ImplicitArgInfoList();
 }
 
-bool ImplicitArgs::isImplicitImage(ImplicitArg::ArgType argType) 
-{ 
+bool ImplicitArgs::isImplicitImage(ImplicitArg::ArgType argType)
+{
     return (argType >= ImplicitArg::IMAGES_START) && (argType <= ImplicitArg::IMAGES_END);
 }
 
-bool ImplicitArgs::isImplicitStruct(ImplicitArg::ArgType argType) 
-{ 
+bool ImplicitArgs::isImplicitStruct(ImplicitArg::ArgType argType)
+{
     return (argType >= ImplicitArg::STRUCT_START) && (argType <= ImplicitArg::STRUCT_END);
 }
 
@@ -564,7 +564,7 @@ int32_t ImplicitArgs::getStructArgOffset(unsigned int index) const
 TODO("Refactor code to avoid code triplication for getArgInFunc(), getImplicitArg() and WIFuncResolution::getImplicitArg()")
 Argument* ImplicitArgs::getArgInFunc(llvm::Function& F, ImplicitArg::ArgType argType) {
     assert(IGCLLVM::GetFuncArgSize(F) >= size() && "Invalid number of argumnents in the function!");
-    
+
     unsigned int argIndex       =  getArgIndex(argType);
     unsigned int argIndexInFunc = IGCLLVM::GetFuncArgSize(F) - size() + argIndex;
     Function::arg_iterator arg  = F.arg_begin();
@@ -591,7 +591,7 @@ Argument* ImplicitArgs::getImplicitArg(llvm::Function& F, ImplicitArg::ArgType a
 Argument* ImplicitArgs::getNumberedImplicitArg(llvm::Function& F, ImplicitArg::ArgType argType, int argNum)
 {
     assert(IGCLLVM::GetFuncArgSize(F) >= size() && "Invalid number of arguments in the function!");
-    
+
     unsigned int numImplicitArgs = size();
     unsigned int implicitArgIndex = this->getNumberedArgIndex(argType, argNum);
     if (implicitArgIndex == numImplicitArgs)

IGC/AdaptorCommon/ImplicitArgs.hpp

@@ -43,7 +43,7 @@ namespace llvm
 
 namespace IGC
 {
-    /// @brief  ImplicitArg is used for representing the implict information that is passed from the 
+    /// @brief  ImplicitArg is used for representing the implict information that is passed from the
     ///         OpenCL runtime to IGC, charecterize it and allow IGC to use it.
     /// @Author Marina Yatsina
     class ImplicitArg {
@@ -126,7 +126,7 @@ namespace IGC
             NUM_IMPLICIT_ARGS
         };
 
-        /// @brief This set type is used to contain function arguments 
+        /// @brief This set type is used to contain function arguments
         typedef std::set<int> ArgValSet;
 
         /// @brief This map type is used to map implicit argument types to
@@ -219,17 +219,17 @@ namespace IGC
         /// @return The implicit argument's index for a given argument type
         unsigned int getArgIndex(ImplicitArg::ArgType argType);
 
-        /// @brief  Returns the index of the appropriate implicit image or sampler argument 
+        /// @brief  Returns the index of the appropriate implicit image or sampler argument
         ///         based on the given argument type and the associated image argument
-        /// @param  argType         The implicit argument type 
+        /// @param  argType         The implicit argument type
         ///                         (height/width/depth for images, normalized/address/snapwa for samplers)
         /// @param  image           The associated image/sampler argument
         /// @return The implicit argument's index for a given argument type
         unsigned int getImageArgIndex(ImplicitArg::ArgType argType, const llvm::Argument* image);
 
-        /// @brief  Returns the index of the appropriate implicit argument 
+        /// @brief  Returns the index of the appropriate implicit argument
         ///         based on the given argument type and the argument number
-        /// @param  argType         The implicit argument type 
+        /// @param  argType         The implicit argument type
         ///                         (height/width/depth for images, normalized/address/snapwa for samplers)
         /// @param  argNum          The explicit argument number of the type
         /// @return The implicit argument's index for a given argument type
@@ -250,7 +250,7 @@ namespace IGC
         /// @return The structure offset of the given implicit argument index
         int32_t getStructArgOffset(uint index) const;
 
-        /// @brief  Creates implict arguments metadata for the give function based on the given implicit arguments 
+        /// @brief  Creates implict arguments metadata for the give function based on the given implicit arguments
         ///         it should receive. If implicit metadata exists, it adds to it.
         /// @param  F               The function for which to create the implicit argument's metadata
         /// @param  implicitArgs    The implicit argument that are required by the given function
@@ -268,7 +268,7 @@ namespace IGC
 
         static void addStructArgs(llvm::Function& F, const llvm::Argument* A, const ImplicitArg::StructArgList& S, IGCMD::MetaDataUtils* pMdUtils);
 
-        /// @brief  Creates implict arguments metadata for the give function based on the given implicit arguments 
+        /// @brief  Creates implict arguments metadata for the give function based on the given implicit arguments
         ///         it should receive. If implicit metadata exists, it adds to it.
         /// @param  F               The function for which to create the implicit argument's metadata
         /// @param  argMap          A map of implict argument types to the set of numbers of arguments
@@ -286,7 +286,7 @@ namespace IGC
         /// @param  argType         The type of the implict argument that should be returned
         /// @return The (implicit) function argument associated with the given argument type
         llvm::Argument* getArgInFunc(llvm::Function& F, ImplicitArg::ArgType argType);
-        
+
         /// @brief  Check if the given implicit argument type exist in the(implicit) function argument associated
         /// @param  argType         The type of the implict argument that should be checked
         /// @return true if the argument exist, false otherwise.