Backout for ROV perf change

Author: fftzeng

IGC/Compiler/CISACodeGen/EmitVISAPass.cpp

@@ -9451,34 +9451,6 @@ void EmitPass::emitUAVSerialize()
     m_encoder->Push();
 }
 
-bool EmitPass::useRasterizerOrderedByteAddressBuffer(GenIntrinsicInst* inst)
-{
-    bool isRov = false;
-    std::vector<uint32_t> ROV_RV = m_currShader->m_ModuleMetadata->RasterizerOrderedByteAddressBuffer;
-
-    unsigned calleeArgNo = 0;
-    PushInfo& pushInfo = m_currShader->m_ModuleMetadata->pushInfo;
-
-    Value* src = IGC::TracePointerSource(inst->getOperand(0));
-    if (Argument * calleeArg = dyn_cast<Argument>(src))
-    {
-        calleeArgNo = calleeArg->getArgNo();
-        for (auto index_it = pushInfo.constantReg.begin(); index_it != pushInfo.constantReg.end(); ++index_it)
-        {
-            if (index_it->second == calleeArgNo)
-            {
-                if (std::find(ROV_RV.begin(), ROV_RV.end(), index_it->first) != ROV_RV.end())
-                {
-                    isRov = true;
-                    break;
-                }
-            }
-        }
-    }
-    return isRov;
-}
-
-
 void EmitPass::emitLoadRawIndexed(GenIntrinsicInst* inst)
 {
     Value* buf_ptrv = inst->getOperand(0);

IGC/Compiler/CISACodeGen/EmitVISAPass.hpp

@@ -179,9 +179,6 @@ class EmitPass : public llvm::FunctionPass
     // set the predicate with current active channels
     void emitPredicateFromChannelIP(CVariable* dst, CVariable* alias = NULL);
 
-    // check if this load/store is accessing RasterizerOrderedByteAddressBuffer
-    bool useRasterizerOrderedByteAddressBuffer(llvm::GenIntrinsicInst* inst);
-
     // Helper methods for message emit functions.
     template <typename T>
     void prepareRenderTargetWritePayload(

IGC/Compiler/CISACodeGen/PixelShaderLowering.cpp

@@ -272,9 +272,7 @@ bool PixelShaderLowering::runOnFunction(llvm::Function& F)
     {
         // Emitting a fence to ensure that the uav write is completed before an EOT is issued
         IRBuilder<> builder(F.getContext());
-
-        bool fenceFlushNone = 0;
-        EmitMemoryFence(builder, fenceFlushNone);
+        EmitMemoryFence(builder);
     }
 
     // EmitRender target write intrinsic
@@ -503,7 +501,7 @@ void PixelShaderLowering::FindIntrinsicOutput(
     }
 }
 
-void PixelShaderLowering::EmitMemoryFence(IRBuilder<>& builder, bool forceFlushNone)
+void PixelShaderLowering::EmitMemoryFence(IRBuilder<>& builder)
 {
     Value* trueValue = builder.getInt1(true);
     Value* falseValue = builder.getInt1(false);
@@ -519,7 +517,7 @@ void PixelShaderLowering::EmitMemoryFence(IRBuilder<>& builder, bool forceFlushN
         falseValue,
     };
 
-    CallInst* temp = GenIntrinsicInst::Create(GenISAIntrinsic::getDeclaration(m_module, GenISAIntrinsic::GenISA_memoryfence),
+    GenIntrinsicInst::Create(GenISAIntrinsic::getDeclaration(m_module, GenISAIntrinsic::GenISA_memoryfence),
         arguments,
         "",
         m_ReturnBlock->getTerminator());

IGC/Compiler/CISACodeGen/PixelShaderLowering.hpp

@@ -113,7 +113,7 @@ class PixelShaderLowering : public llvm::FunctionPass
         llvm::Value*& mask, llvm::Value*& src0Alpha,
         DebugLocArray& debugLocs);
 
-    void EmitMemoryFence(llvm::IRBuilder<>& builder, bool forceFlushNone = 0);
+    void EmitMemoryFence(llvm::IRBuilder<>& builder);
     void EmitRTWrite(ColorOutputArray& color, llvm::Value* depth,
         llvm::Value* stencil, llvm::Value* mask,
         llvm::Value* src0Alpha, DebugLocArray& debugLocs);

IGC/common/MDFrameWork.h

@@ -449,7 +449,6 @@ namespace IGC
         std::vector<InlineProgramScopeBuffer> inlineGlobalBuffers;
         std::vector<PointerProgramBinaryInfo> GlobalPointerProgramBinaryInfos;
         std::vector<PointerProgramBinaryInfo> ConstantPointerProgramBinaryInfos;
-        std::vector<uint32_t> RasterizerOrderedByteAddressBuffer;
         unsigned int MinNOSPushConstantSize = 0;
         std::map<llvm::GlobalVariable*, int> inlineProgramScopeOffsets;
         ShaderData shaderData;