From 98d0201577ba21223e6d9a54b1240fe49524d486 Mon Sep 17 00:00:00 2001 From: Alok Hota Date: Fri, 25 May 2018 10:19:46 -0500 Subject: [PATCH] swr/rast: Renamed MetaData calls Reviewed-by: Bruce Cherniak --- .../drivers/swr/rasterizer/jitter/builder.cpp | 170 ++++++++++----------- .../drivers/swr/rasterizer/jitter/builder.h | 4 +- 2 files changed, 87 insertions(+), 87 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/jitter/builder.cpp b/src/gallium/drivers/swr/rasterizer/jitter/builder.cpp index e1c5d80..4b06aaa 100644 --- a/src/gallium/drivers/swr/rasterizer/jitter/builder.cpp +++ b/src/gallium/drivers/swr/rasterizer/jitter/builder.cpp @@ -1,32 +1,32 @@ /**************************************************************************** -* Copyright (C) 2014-2015 Intel Corporation. All Rights Reserved. -* -* Permission is hereby granted, free of charge, to any person obtaining a -* copy of this software and associated documentation files (the "Software"), -* to deal in the Software without restriction, including without limitation -* the rights to use, copy, modify, merge, publish, distribute, sublicense, -* and/or sell copies of the Software, and to permit persons to whom the -* Software is furnished to do so, subject to the following conditions: -* -* The above copyright notice and this permission notice (including the next -* paragraph) shall be included in all copies or substantial portions of the -* Software. -* -* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR -* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, -* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL -* THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER -* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING -* FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS -* IN THE SOFTWARE. -* -* @file builder.h -* -* @brief Includes all the builder related functionality -* -* Notes: -* -******************************************************************************/ + * Copyright (C) 2014-2015 Intel Corporation. All Rights Reserved. + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS + * IN THE SOFTWARE. + * + * @file builder.h + * + * @brief Includes all the builder related functionality + * + * Notes: + * + ******************************************************************************/ #include "jit_pch.hpp" #include "builder.h" @@ -38,11 +38,9 @@ namespace SwrJit ////////////////////////////////////////////////////////////////////////// /// @brief Contructor for Builder. /// @param pJitMgr - JitManager which contains modules, function passes, etc. - Builder::Builder(JitManager *pJitMgr) - : mpJitMgr(pJitMgr), - mpPrivateContext(nullptr) + Builder::Builder(JitManager *pJitMgr) : mpJitMgr(pJitMgr), mpPrivateContext(nullptr) { - mVWidth = pJitMgr->mVWidth; + mVWidth = pJitMgr->mVWidth; mVWidth16 = 16; mpIRBuilder = &pJitMgr->mBuilder; @@ -70,29 +68,29 @@ namespace SwrJit // Built in types: simd16 - mSimd16Int1Ty = VectorType::get(mInt1Ty, mVWidth16); - mSimd16Int16Ty = VectorType::get(mInt16Ty, mVWidth16); - mSimd16Int32Ty = VectorType::get(mInt32Ty, mVWidth16); - mSimd16Int64Ty = VectorType::get(mInt64Ty, mVWidth16); - mSimd16FP16Ty = VectorType::get(mFP16Ty, mVWidth16); - mSimd16FP32Ty = VectorType::get(mFP32Ty, mVWidth16); - mSimd16VectorTy = ArrayType::get(mSimd16FP32Ty, 4); - mSimd16VectorTRTy = ArrayType::get(mSimd16FP32Ty, 5); + mSimd16Int1Ty = VectorType::get(mInt1Ty, mVWidth16); + mSimd16Int16Ty = VectorType::get(mInt16Ty, mVWidth16); + mSimd16Int32Ty = VectorType::get(mInt32Ty, mVWidth16); + mSimd16Int64Ty = VectorType::get(mInt64Ty, mVWidth16); + mSimd16FP16Ty = VectorType::get(mFP16Ty, mVWidth16); + mSimd16FP32Ty = VectorType::get(mFP32Ty, mVWidth16); + mSimd16VectorTy = ArrayType::get(mSimd16FP32Ty, 4); + mSimd16VectorTRTy = ArrayType::get(mSimd16FP32Ty, 5); - mSimd32Int8Ty = VectorType::get(mInt8Ty, 32); + mSimd32Int8Ty = VectorType::get(mInt8Ty, 32); - if (sizeof(uint32_t*) == 4) + if (sizeof(uint32_t *) == 4) { - mIntPtrTy = mInt32Ty; - mSimdIntPtrTy = mSimdInt32Ty; + mIntPtrTy = mInt32Ty; + mSimdIntPtrTy = mSimdInt32Ty; mSimd16IntPtrTy = mSimd16Int32Ty; } else { - SWR_ASSERT(sizeof(uint32_t*) == 8); + SWR_ASSERT(sizeof(uint32_t *) == 8); - mIntPtrTy = mInt64Ty; - mSimdIntPtrTy = mSimdInt64Ty; + mIntPtrTy = mInt64Ty; + mSimdIntPtrTy = mSimdInt64Ty; mSimd16IntPtrTy = mSimd16Int64Ty; } } @@ -101,53 +99,53 @@ namespace SwrJit { mVWidth = width; - mSimdInt1Ty = VectorType::get(mInt1Ty, mVWidth); - mSimdInt16Ty = VectorType::get(mInt16Ty, mVWidth); - mSimdInt32Ty = VectorType::get(mInt32Ty, mVWidth); - mSimdInt64Ty = VectorType::get(mInt64Ty, mVWidth); - mSimdFP16Ty = VectorType::get(mFP16Ty, mVWidth); - mSimdFP32Ty = VectorType::get(mFP32Ty, mVWidth); - mSimdVectorTy = ArrayType::get(mSimdFP32Ty, 4); + mSimdInt1Ty = VectorType::get(mInt1Ty, mVWidth); + mSimdInt16Ty = VectorType::get(mInt16Ty, mVWidth); + mSimdInt32Ty = VectorType::get(mInt32Ty, mVWidth); + mSimdInt64Ty = VectorType::get(mInt64Ty, mVWidth); + mSimdFP16Ty = VectorType::get(mFP16Ty, mVWidth); + mSimdFP32Ty = VectorType::get(mFP32Ty, mVWidth); + mSimdVectorTy = ArrayType::get(mSimdFP32Ty, 4); mSimdVectorIntTy = ArrayType::get(mSimdInt32Ty, 4); - mSimdVectorTRTy = ArrayType::get(mSimdFP32Ty, 5); + mSimdVectorTRTy = ArrayType::get(mSimdFP32Ty, 5); } /// @brief Mark this alloca as temporary to avoid hoisting later on - void Builder::SetTempAlloca(Value* inst) + void Builder::SetTempAlloca(Value *inst) { - AllocaInst* pAlloca = dyn_cast(inst); + AllocaInst *pAlloca = dyn_cast(inst); SWR_ASSERT(pAlloca, "Unexpected non-alloca instruction"); - MDNode* N = MDNode::get(JM()->mContext, MDString::get(JM()->mContext, "is_temp_alloca")); + MDNode *N = MDNode::get(JM()->mContext, MDString::get(JM()->mContext, "is_temp_alloca")); pAlloca->setMetadata("is_temp_alloca", N); } - bool Builder::IsTempAlloca(Value* inst) + bool Builder::IsTempAlloca(Value *inst) { - AllocaInst* pAlloca = dyn_cast(inst); + AllocaInst *pAlloca = dyn_cast(inst); SWR_ASSERT(pAlloca, "Unexpected non-alloca instruction"); return (pAlloca->getMetadata("is_temp_alloca") != nullptr); } - // Returns true if able to find an intrinsic to mark - bool Builder::SetTexelMaskEvaluate(Instruction* inst) + // Returns true if able to find a call instruction to mark + bool Builder::SetNamedMetaDataOnCallInstr(Instruction *inst, StringRef mdName) { - CallInst* pGenIntrin = dyn_cast(inst); - if (pGenIntrin) + CallInst *pCallInstr = dyn_cast(inst); + if (pCallInstr) { - MDNode* N = MDNode::get(JM()->mContext, MDString::get(JM()->mContext, "is_evaluate")); - pGenIntrin->setMetadata("is_evaluate", N); + MDNode *N = MDNode::get(JM()->mContext, MDString::get(JM()->mContext, mdName)); + pCallInstr->setMetadata(mdName, N); return true; } else { // Follow use def chain back up - for (Use& u : inst->operands()) + for (Use &u : inst->operands()) { - Instruction* srcInst = dyn_cast(u.get()); + Instruction *srcInst = dyn_cast(u.get()); if (srcInst) { - if (SetTexelMaskEvaluate(srcInst)) + if (SetNamedMetaDataOnCallInstr(srcInst, mdName)) { return true; } @@ -158,21 +156,22 @@ namespace SwrJit return false; } - bool Builder::IsTexelMaskEvaluate(Instruction* genSampleOrLoadIntrinsic) + bool Builder::HasNamedMetaDataOnCallInstr(Instruction *inst, + StringRef mdName) { - CallInst* pGenIntrin = dyn_cast(genSampleOrLoadIntrinsic); + CallInst *pCallInstr = dyn_cast(inst); - if (!pGenIntrin) + if (!pCallInstr) { return false; } - return (pGenIntrin->getMetadata("is_evaluate") != nullptr); + return (pCallInstr->getMetadata(mdName) != nullptr); } ////////////////////////////////////////////////////////////////////////// /// @brief Packetizes the type. Assumes SOA conversion. - Type* Builder::GetVectorType(Type* pType) + Type *Builder::GetVectorType(Type *pType) { if (pType->isVectorTy()) { @@ -182,36 +181,37 @@ namespace SwrJit // [N x float] should packetize to [N x <8 x float>] if (pType->isArrayTy()) { - uint32_t arraySize = pType->getArrayNumElements(); - Type* pArrayType = pType->getArrayElementType(); - Type* pVecArrayType = GetVectorType(pArrayType); - Type* pVecType = ArrayType::get(pVecArrayType, arraySize); + uint32_t arraySize = pType->getArrayNumElements(); + Type * pArrayType = pType->getArrayElementType(); + Type * pVecArrayType = GetVectorType(pArrayType); + Type * pVecType = ArrayType::get(pVecArrayType, arraySize); return pVecType; } // {float,int} should packetize to {<8 x float>, <8 x int>} if (pType->isAggregateType()) { - uint32_t numElems = pType->getStructNumElements(); - SmallVector vecTypes; + uint32_t numElems = pType->getStructNumElements(); + SmallVector vecTypes; for (uint32_t i = 0; i < numElems; ++i) { - Type* pElemType = pType->getStructElementType(i); - Type* pVecElemType = GetVectorType(pElemType); + Type *pElemType = pType->getStructElementType(i); + Type *pVecElemType = GetVectorType(pElemType); vecTypes.push_back(pVecElemType); } - Type* pVecType = StructType::get(JM()->mContext, vecTypes); + Type *pVecType = StructType::get(JM()->mContext, vecTypes); return pVecType; } // [N x float]* should packetize to [N x <8 x float>]* if (pType->isPointerTy() && pType->getPointerElementType()->isArrayTy()) { - return PointerType::get(GetVectorType(pType->getPointerElementType()), pType->getPointerAddressSpace()); + return PointerType::get(GetVectorType(pType->getPointerElementType()), + pType->getPointerAddressSpace()); } // should packetize to <8 x > - Type* vecType = VectorType::get(pType, JM()->mVWidth); + Type *vecType = VectorType::get(pType, JM()->mVWidth); return vecType; } } diff --git a/src/gallium/drivers/swr/rasterizer/jitter/builder.h b/src/gallium/drivers/swr/rasterizer/jitter/builder.h index 08a3a6e..c49d07e 100644 --- a/src/gallium/drivers/swr/rasterizer/jitter/builder.h +++ b/src/gallium/drivers/swr/rasterizer/jitter/builder.h @@ -121,8 +121,8 @@ namespace SwrJit void SetTargetWidth(uint32_t width); void SetTempAlloca(Value* inst); bool IsTempAlloca(Value* inst); - bool SetTexelMaskEvaluate(Instruction* inst); - bool IsTexelMaskEvaluate(Instruction* inst); + bool SetNamedMetaDataOnCallInstr(Instruction* inst, StringRef mdName); + bool HasNamedMetaDataOnCallInstr(Instruction* inst, StringRef mdName); Type* GetVectorType(Type* pType); void SetMetadata(StringRef s, uint32_t val) { -- 2.7.4