| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053105410551056105710581059106010611062106310641065106610671068106910701071107210731074107510761077107810791080108110821083108410851086108710881089109010911092109310941095109610971098109911001101110211031104110511061107110811091110111111121113111411151116111711181119112011211122112311241125112611271128112911301131113211331134113511361137113811391140114111421143114411451146114711481149115011511152115311541155115611571158115911601161116211631164116511661167116811691170117111721173117411751176117711781179118011811182118311841185118611871188118911901191119211931194119511961197119811991200120112021203120412051206 |
- ///////////////////////////////////////////////////////////////////////////////
- // //
- // DxilPreparePasses.cpp //
- // Copyright (C) Microsoft Corporation. All rights reserved. //
- // This file is distributed under the University of Illinois Open Source //
- // License. See LICENSE.TXT for details. //
- // //
- // Passes to prepare DxilModule. //
- // //
- ///////////////////////////////////////////////////////////////////////////////
- #include "dxc/HLSL/DxilGenerationPass.h"
- #include "dxc/DXIL/DxilOperations.h"
- #include "dxc/HLSL/HLOperations.h"
- #include "dxc/DXIL/DxilModule.h"
- #include "dxc/Support/Global.h"
- #include "dxc/DXIL/DxilTypeSystem.h"
- #include "dxc/DXIL/DxilUtil.h"
- #include "dxc/DXIL/DxilFunctionProps.h"
- #include "dxc/DXIL/DxilInstructions.h"
- #include "dxc/DXIL/DxilConstants.h"
- #include "dxc/HlslIntrinsicOp.h"
- #include "llvm/IR/GetElementPtrTypeIterator.h"
- #include "llvm/IR/IRBuilder.h"
- #include "llvm/IR/Instructions.h"
- #include "llvm/IR/InstIterator.h"
- #include "llvm/IR/IntrinsicInst.h"
- #include "llvm/Analysis/PostDominators.h"
- #include "llvm/IR/Module.h"
- #include "llvm/IR/DebugInfo.h"
- #include "llvm/IR/PassManager.h"
- #include "llvm/ADT/BitVector.h"
- #include "llvm/Pass.h"
- #include "llvm/Transforms/Utils/Local.h"
- #include "llvm/Analysis/AssumptionCache.h"
- #include "llvm/Analysis/DxilValueCache.h"
- #include "llvm/Analysis/LoopInfo.h"
- #include <memory>
- #include <unordered_set>
- using namespace llvm;
- using namespace hlsl;
- namespace {
- class InvalidateUndefResources : public ModulePass {
- public:
- static char ID;
- explicit InvalidateUndefResources() : ModulePass(ID) {
- initializeScalarizerPass(*PassRegistry::getPassRegistry());
- }
- const char *getPassName() const override { return "Invalidate undef resources"; }
- bool runOnModule(Module &M) override;
- };
- }
- char InvalidateUndefResources::ID = 0;
- ModulePass *llvm::createInvalidateUndefResourcesPass() { return new InvalidateUndefResources(); }
- INITIALIZE_PASS(InvalidateUndefResources, "invalidate-undef-resource", "Invalidate undef resources", false, false)
- bool InvalidateUndefResources::runOnModule(Module &M) {
- // Undef resources typically indicate uninitialized locals being used
- // in some code path, which we should catch and report. However, some
- // code patterns in large shaders cause dead undef resources to momentarily,
- // which is not an error. We must wait until cleanup passes
- // have run to know whether we must produce an error.
- // However, we can't leave the undef values in because they could eliminated,
- // such as by reading from resources seen in a code path that was not taken.
- // We avoid the problem by replacing undef values by another invalid
- // value that we can identify later.
- for (auto &F : M.functions()) {
- if (GetHLOpcodeGroupByName(&F) == HLOpcodeGroup::HLCreateHandle) {
- Type *ResTy = F.getFunctionType()->getParamType(
- HLOperandIndex::kCreateHandleResourceOpIdx);
- UndefValue *UndefRes = UndefValue::get(ResTy);
- if (!UndefRes->use_empty()) {
- Constant *InvalidRes = ConstantAggregateZero::get(ResTy);
- UndefRes->replaceAllUsesWith(InvalidRes);
- }
- }
- }
- return false;
- }
- ///////////////////////////////////////////////////////////////////////////////
- namespace {
- class SimplifyInst : public FunctionPass {
- public:
- static char ID;
- SimplifyInst() : FunctionPass(ID) {
- initializeScalarizerPass(*PassRegistry::getPassRegistry());
- }
- bool runOnFunction(Function &F) override;
- private:
- };
- }
- char SimplifyInst::ID = 0;
- FunctionPass *llvm::createSimplifyInstPass() { return new SimplifyInst(); }
- INITIALIZE_PASS(SimplifyInst, "simplify-inst", "Simplify Instructions", false, false)
- bool SimplifyInst::runOnFunction(Function &F) {
- for (Function::iterator BBI = F.begin(), BBE = F.end(); BBI != BBE; ++BBI) {
- BasicBlock *BB = BBI;
- llvm::SimplifyInstructionsInBlock(BB, nullptr);
- }
- return true;
- }
- ///////////////////////////////////////////////////////////////////////////////
- namespace {
- class DxilDeadFunctionElimination : public ModulePass {
- public:
- static char ID; // Pass identification, replacement for typeid
- explicit DxilDeadFunctionElimination () : ModulePass(ID) {}
- const char *getPassName() const override { return "Remove all unused function except entry from DxilModule"; }
- bool runOnModule(Module &M) override {
- if (M.HasDxilModule()) {
- DxilModule &DM = M.GetDxilModule();
- bool IsLib = DM.GetShaderModel()->IsLib();
- // Remove unused functions except entry and patch constant func.
- // For library profile, only remove unused external functions.
- Function *EntryFunc = DM.GetEntryFunction();
- Function *PatchConstantFunc = DM.GetPatchConstantFunction();
- return dxilutil::RemoveUnusedFunctions(M, EntryFunc, PatchConstantFunc,
- IsLib);
- }
- return false;
- }
- };
- }
- char DxilDeadFunctionElimination::ID = 0;
- ModulePass *llvm::createDxilDeadFunctionEliminationPass() {
- return new DxilDeadFunctionElimination();
- }
- INITIALIZE_PASS(DxilDeadFunctionElimination, "dxil-dfe", "Remove all unused function except entry from DxilModule", false, false)
- ///////////////////////////////////////////////////////////////////////////////
- bool CleanupSharedMemoryAddrSpaceCast(Module &M);
- namespace {
- static void TransferEntryFunctionAttributes(Function *F, Function *NewFunc) {
- // Keep necessary function attributes
- AttributeSet attributeSet = F->getAttributes();
- StringRef attrKind, attrValue;
- if (attributeSet.hasAttribute(AttributeSet::FunctionIndex, DXIL::kFP32DenormKindString)) {
- Attribute attribute = attributeSet.getAttribute(AttributeSet::FunctionIndex, DXIL::kFP32DenormKindString);
- DXASSERT(attribute.isStringAttribute(), "otherwise we have wrong fp-denorm-mode attribute.");
- attrKind = attribute.getKindAsString();
- attrValue = attribute.getValueAsString();
- }
- if (F == NewFunc) {
- NewFunc->removeAttributes(AttributeSet::FunctionIndex, attributeSet);
- }
- if (!attrKind.empty() && !attrValue.empty())
- NewFunc->addFnAttr(attrKind, attrValue);
- }
- // If this returns non-null, the old function F has been stripped and can be deleted.
- static Function *StripFunctionParameter(Function *F, DxilModule &DM,
- DenseMap<const Function *, DISubprogram *> &FunctionDIs) {
- if (F->arg_empty() && F->getReturnType()->isVoidTy()) {
- // This will strip non-entry function attributes
- TransferEntryFunctionAttributes(F, F);
- return nullptr;
- }
- Module &M = *DM.GetModule();
- Type *VoidTy = Type::getVoidTy(M.getContext());
- FunctionType *FT = FunctionType::get(VoidTy, false);
- for (auto &arg : F->args()) {
- if (!arg.user_empty())
- return nullptr;
- DbgDeclareInst *DDI = llvm::FindAllocaDbgDeclare(&arg);
- if (DDI) {
- DDI->eraseFromParent();
- }
- }
- Function *NewFunc = Function::Create(FT, F->getLinkage());
- M.getFunctionList().insert(F, NewFunc);
- // Splice the body of the old function right into the new function.
- NewFunc->getBasicBlockList().splice(NewFunc->begin(), F->getBasicBlockList());
- TransferEntryFunctionAttributes(F, NewFunc);
- // Patch the pointer to LLVM function in debug info descriptor.
- auto DI = FunctionDIs.find(F);
- if (DI != FunctionDIs.end()) {
- DISubprogram *SP = DI->second;
- SP->replaceFunction(NewFunc);
- // Ensure the map is updated so it can be reused on subsequent argument
- // promotions of the same function.
- FunctionDIs.erase(DI);
- FunctionDIs[NewFunc] = SP;
- }
- NewFunc->takeName(F);
- if (DM.HasDxilFunctionProps(F)) {
- DM.ReplaceDxilEntryProps(F, NewFunc);
- }
- DM.GetTypeSystem().EraseFunctionAnnotation(F);
- DM.GetTypeSystem().AddFunctionAnnotation(NewFunc);
- return NewFunc;
- }
- void CheckInBoundForTGSM(GlobalVariable &GV, const DataLayout &DL) {
- for (User *U : GV.users()) {
- if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(U)) {
- bool allImmIndex = true;
- for (auto Idx = GEP->idx_begin(), E = GEP->idx_end(); Idx != E; Idx++) {
- if (!isa<ConstantInt>(Idx)) {
- allImmIndex = false;
- break;
- }
- }
- if (!allImmIndex)
- GEP->setIsInBounds(false);
- else {
- Value *Ptr = GEP->getPointerOperand();
- unsigned size =
- DL.getTypeAllocSize(Ptr->getType()->getPointerElementType());
- unsigned valSize =
- DL.getTypeAllocSize(GEP->getType()->getPointerElementType());
- SmallVector<Value *, 8> Indices(GEP->idx_begin(), GEP->idx_end());
- unsigned offset =
- DL.getIndexedOffset(GEP->getPointerOperandType(), Indices);
- if ((offset + valSize) > size)
- GEP->setIsInBounds(false);
- }
- }
- }
- }
- static bool GetUnsignedVal(Value *V, uint32_t *pValue) {
- ConstantInt *CI = dyn_cast<ConstantInt>(V);
- if (!CI) return false;
- uint64_t u = CI->getZExtValue();
- if (u > UINT32_MAX) return false;
- *pValue = (uint32_t)u;
- return true;
- }
- static void MarkUsedSignatureElements(Function *F, DxilModule &DM) {
- DXASSERT_NOMSG(F != nullptr);
- // For every loadInput/storeOutput, update the corresponding ReadWriteMask.
- // F is a pointer to a Function instance
- for (llvm::inst_iterator I = llvm::inst_begin(F), E = llvm::inst_end(F); I != E; ++I) {
- DxilInst_LoadInput LI(&*I);
- DxilInst_StoreOutput SO(&*I);
- DxilInst_LoadPatchConstant LPC(&*I);
- DxilInst_StorePatchConstant SPC(&*I);
- DxilInst_StoreVertexOutput SVO(&*I);
- DxilInst_StorePrimitiveOutput SPO(&*I);
- DxilSignature *pSig;
- uint32_t col, row, sigId;
- bool bDynIdx = false;
- if (LI) {
- if (!GetUnsignedVal(LI.get_inputSigId(), &sigId)) continue;
- if (!GetUnsignedVal(LI.get_colIndex(), &col)) continue;
- if (!GetUnsignedVal(LI.get_rowIndex(), &row)) bDynIdx = true;
- pSig = &DM.GetInputSignature();
- }
- else if (SO) {
- if (!GetUnsignedVal(SO.get_outputSigId(), &sigId)) continue;
- if (!GetUnsignedVal(SO.get_colIndex(), &col)) continue;
- if (!GetUnsignedVal(SO.get_rowIndex(), &row)) bDynIdx = true;
- pSig = &DM.GetOutputSignature();
- }
- else if (SPC) {
- if (!GetUnsignedVal(SPC.get_outputSigID(), &sigId)) continue;
- if (!GetUnsignedVal(SPC.get_col(), &col)) continue;
- if (!GetUnsignedVal(SPC.get_row(), &row)) bDynIdx = true;
- pSig = &DM.GetPatchConstOrPrimSignature();
- }
- else if (LPC) {
- if (!GetUnsignedVal(LPC.get_inputSigId(), &sigId)) continue;
- if (!GetUnsignedVal(LPC.get_col(), &col)) continue;
- if (!GetUnsignedVal(LPC.get_row(), &row)) bDynIdx = true;
- pSig = &DM.GetPatchConstOrPrimSignature();
- }
- else if (SVO) {
- if (!GetUnsignedVal(SVO.get_outputSigId(), &sigId)) continue;
- if (!GetUnsignedVal(SVO.get_colIndex(), &col)) continue;
- if (!GetUnsignedVal(SVO.get_rowIndex(), &row)) bDynIdx = true;
- pSig = &DM.GetOutputSignature();
- }
- else if (SPO) {
- if (!GetUnsignedVal(SPO.get_outputSigId(), &sigId)) continue;
- if (!GetUnsignedVal(SPO.get_colIndex(), &col)) continue;
- if (!GetUnsignedVal(SPO.get_rowIndex(), &row)) bDynIdx = true;
- pSig = &DM.GetPatchConstOrPrimSignature();
- }
- else {
- continue;
- }
- // Consider being more fine-grained about masks.
- // We report sometimes-read on input as always-read.
- auto &El = pSig->GetElement(sigId);
- unsigned UsageMask = El.GetUsageMask();
- unsigned colBit = 1 << col;
- if (!(colBit & UsageMask)) {
- El.SetUsageMask(UsageMask | colBit);
- }
- if (bDynIdx && (El.GetDynIdxCompMask() & colBit) == 0) {
- El.SetDynIdxCompMask(El.GetDynIdxCompMask() | colBit);
- }
- }
- }
- class DxilFinalizeModule : public ModulePass {
- public:
- static char ID; // Pass identification, replacement for typeid
- explicit DxilFinalizeModule() : ModulePass(ID) {}
- const char *getPassName() const override { return "HLSL DXIL Finalize Module"; }
- void patchValidation_1_1(Module &M) {
- for (iplist<Function>::iterator F : M.getFunctionList()) {
- for (Function::iterator BBI = F->begin(), BBE = F->end(); BBI != BBE;
- ++BBI) {
- BasicBlock *BB = BBI;
- for (BasicBlock::iterator II = BB->begin(), IE = BB->end(); II != IE;
- ++II) {
- Instruction *I = II;
- if (I->hasMetadataOtherThanDebugLoc()) {
- SmallVector<std::pair<unsigned, MDNode*>, 2> MDs;
- I->getAllMetadataOtherThanDebugLoc(MDs);
- for (auto &MD : MDs) {
- unsigned kind = MD.first;
- // Remove Metadata which validation_1_0 not allowed.
- bool bNeedPatch = kind == LLVMContext::MD_tbaa ||
- kind == LLVMContext::MD_prof ||
- (kind > LLVMContext::MD_fpmath &&
- kind <= LLVMContext::MD_dereferenceable_or_null);
- if (bNeedPatch)
- I->setMetadata(kind, nullptr);
- }
- }
- }
- }
- }
- }
- void patchDxil_1_6(Module &M, hlsl::OP *hlslOP) {
- for (auto it : hlslOP->GetOpFuncList(DXIL::OpCode::AnnotateHandle)) {
- Function *F = it.second;
- if (!F)
- continue;
- for (auto uit = F->user_begin(); uit != F->user_end();) {
- CallInst *CI = cast<CallInst>(*(uit++));
- DxilInst_AnnotateHandle annoteHdl(CI);
- Value *hdl = annoteHdl.get_res();
- CI->replaceAllUsesWith(hdl);
- CI->eraseFromParent();
- }
- }
- }
- bool runOnModule(Module &M) override {
- if (M.HasDxilModule()) {
- DxilModule &DM = M.GetDxilModule();
- unsigned ValMajor = 0;
- unsigned ValMinor = 0;
- M.GetDxilModule().GetValidatorVersion(ValMajor, ValMinor);
- unsigned DxilMajor = 0;
- unsigned DxilMinor = 0;
- M.GetDxilModule().GetDxilVersion(DxilMajor, DxilMinor);
- bool IsLib = DM.GetShaderModel()->IsLib();
- // Skip validation patch for lib.
- if (!IsLib) {
- if (ValMajor == 1 && ValMinor <= 1) {
- patchValidation_1_1(M);
- }
- // Set used masks for signature elements
- MarkUsedSignatureElements(DM.GetEntryFunction(), DM);
- if (DM.GetShaderModel()->IsHS())
- MarkUsedSignatureElements(DM.GetPatchConstantFunction(), DM);
- }
- // Remove store undef output.
- hlsl::OP *hlslOP = M.GetDxilModule().GetOP();
- if (DxilMinor < 6) {
- patchDxil_1_6(M, hlslOP);
- }
- RemoveStoreUndefOutput(M, hlslOP);
- // Turn dx.break() conditional into global
- LowerDxBreak(M);
- RemoveUnusedStaticGlobal(M);
- // Remove unnecessary address space casts.
- CleanupSharedMemoryAddrSpaceCast(M);
- // Clear inbound for GEP which has none-const index.
- LegalizeSharedMemoryGEPInbound(M);
- // Strip parameters of entry function.
- StripEntryParameters(M, DM, IsLib);
- // Update flags to reflect any changes.
- DM.CollectShaderFlagsForModule();
- // Update Validator Version
- DM.UpgradeToMinValidatorVersion();
- // Clear intermediate options that shouldn't be in the final DXIL
- DM.ClearIntermediateOptions();
- // Remove unused AllocateRayQuery calls
- RemoveUnusedRayQuery(M);
- if (IsLib && DXIL::CompareVersions(ValMajor, ValMinor, 1, 4) <= 0) {
- // 1.4 validator requires function annotations for all functions
- AddFunctionAnnotationForInitializers(M, DM);
- }
- return true;
- }
- return false;
- }
- private:
- void RemoveUnusedStaticGlobal(Module &M) {
- // Remove unused internal global.
- std::vector<GlobalVariable *> staticGVs;
- for (GlobalVariable &GV : M.globals()) {
- if (dxilutil::IsStaticGlobal(&GV) ||
- dxilutil::IsSharedMemoryGlobal(&GV)) {
- staticGVs.emplace_back(&GV);
- }
- }
- for (GlobalVariable *GV : staticGVs) {
- bool onlyStoreUse = true;
- for (User *user : GV->users()) {
- if (isa<StoreInst>(user))
- continue;
- if (isa<ConstantExpr>(user) && user->user_empty())
- continue;
- onlyStoreUse = false;
- break;
- }
- if (onlyStoreUse) {
- for (auto UserIt = GV->user_begin(); UserIt != GV->user_end();) {
- Value *User = *(UserIt++);
- if (Instruction *I = dyn_cast<Instruction>(User)) {
- I->eraseFromParent();
- } else {
- ConstantExpr *CE = cast<ConstantExpr>(User);
- CE->dropAllReferences();
- }
- }
- GV->eraseFromParent();
- }
- }
- }
- void RemoveStoreUndefOutput(Module &M, hlsl::OP *hlslOP) {
- for (iplist<Function>::iterator F : M.getFunctionList()) {
- if (!hlslOP->IsDxilOpFunc(F))
- continue;
- DXIL::OpCodeClass opClass;
- bool bHasOpClass = hlslOP->GetOpCodeClass(F, opClass);
- DXASSERT_LOCALVAR(bHasOpClass, bHasOpClass, "else not a dxil op func");
- if (opClass != DXIL::OpCodeClass::StoreOutput)
- continue;
- for (auto it = F->user_begin(); it != F->user_end();) {
- CallInst *CI = dyn_cast<CallInst>(*(it++));
- if (!CI)
- continue;
- Value *V = CI->getArgOperand(DXIL::OperandIndex::kStoreOutputValOpIdx);
- // Remove the store of undef.
- if (isa<UndefValue>(V))
- CI->eraseFromParent();
- }
- }
- }
- void LegalizeSharedMemoryGEPInbound(Module &M) {
- const DataLayout &DL = M.getDataLayout();
- // Clear inbound for GEP which has none-const index.
- for (GlobalVariable &GV : M.globals()) {
- if (dxilutil::IsSharedMemoryGlobal(&GV)) {
- CheckInBoundForTGSM(GV, DL);
- }
- }
- }
- void StripEntryParameters(Module &M, DxilModule &DM, bool IsLib) {
- DenseMap<const Function *, DISubprogram *> FunctionDIs =
- makeSubprogramMap(M);
- // Strip parameters of entry function.
- if (!IsLib) {
- if (Function *OldPatchConstantFunc = DM.GetPatchConstantFunction()) {
- Function *NewPatchConstantFunc =
- StripFunctionParameter(OldPatchConstantFunc, DM, FunctionDIs);
- if (NewPatchConstantFunc) {
- DM.SetPatchConstantFunction(NewPatchConstantFunc);
- // Erase once the DxilModule doesn't track the old function anymore
- DXASSERT(DM.IsPatchConstantShader(NewPatchConstantFunc) && !DM.IsPatchConstantShader(OldPatchConstantFunc),
- "Error while migrating to parameter-stripped patch constant function.");
- OldPatchConstantFunc->eraseFromParent();
- }
- }
- if (Function *OldEntryFunc = DM.GetEntryFunction()) {
- StringRef Name = DM.GetEntryFunctionName();
- OldEntryFunc->setName(Name);
- Function *NewEntryFunc = StripFunctionParameter(OldEntryFunc, DM, FunctionDIs);
- if (NewEntryFunc) {
- DM.SetEntryFunction(NewEntryFunc);
- OldEntryFunc->eraseFromParent();
- }
- }
- } else {
- std::vector<Function *> entries;
- // Handle when multiple hull shaders point to the same patch constant function
- MapVector<Function*, llvm::SmallVector<Function*, 2>> PatchConstantFuncUsers;
- for (iplist<Function>::iterator F : M.getFunctionList()) {
- if (DM.IsEntryThatUsesSignatures(F)) {
- auto *FT = F->getFunctionType();
- // Only do this when has parameters.
- if (FT->getNumParams() > 0 || !FT->getReturnType()->isVoidTy()) {
- entries.emplace_back(F);
- }
- DxilFunctionProps& props = DM.GetDxilFunctionProps(F);
- if (props.IsHS() && props.ShaderProps.HS.patchConstantFunc) {
- FunctionType* PatchConstantFuncTy = props.ShaderProps.HS.patchConstantFunc->getFunctionType();
- if (PatchConstantFuncTy->getNumParams() > 0 || !PatchConstantFuncTy->getReturnType()->isVoidTy()) {
- // Accumulate all hull shaders using a given patch constant function,
- // so we can update it once and fix all hull shaders, without having an intermediary
- // state where some hull shaders point to a destroyed patch constant function.
- PatchConstantFuncUsers[props.ShaderProps.HS.patchConstantFunc].emplace_back(F);
- }
- }
- }
- }
- // Strip patch constant functions first
- for (auto &PatchConstantFuncEntry : PatchConstantFuncUsers) {
- Function* OldPatchConstantFunc = PatchConstantFuncEntry.first;
- Function* NewPatchConstantFunc = StripFunctionParameter(OldPatchConstantFunc, DM, FunctionDIs);
- if (NewPatchConstantFunc) {
- // Update all user hull shaders
- for (Function *HullShaderFunc : PatchConstantFuncEntry.second)
- DM.SetPatchConstantFunctionForHS(HullShaderFunc, NewPatchConstantFunc);
- // Erase once the DxilModule doesn't track the old function anymore
- DXASSERT(DM.IsPatchConstantShader(NewPatchConstantFunc) && !DM.IsPatchConstantShader(OldPatchConstantFunc),
- "Error while migrating to parameter-stripped patch constant function.");
- OldPatchConstantFunc->eraseFromParent();
- }
- }
- for (Function *OldEntry : entries) {
- Function *NewEntry = StripFunctionParameter(OldEntry, DM, FunctionDIs);
- if (NewEntry) OldEntry->eraseFromParent();
- }
- }
- }
- void AddFunctionAnnotationForInitializers(Module &M, DxilModule &DM) {
- if (GlobalVariable *GV = M.getGlobalVariable("llvm.global_ctors")) {
- if (isa<ConstantAggregateZero>(GV->getInitializer())) {
- DXASSERT_NOMSG(GV->user_empty());
- GV->eraseFromParent();
- return;
- }
- ConstantArray *init = cast<ConstantArray>(GV->getInitializer());
- for (auto V : init->operand_values()) {
- if (isa<ConstantAggregateZero>(V))
- continue;
- ConstantStruct *CS = cast<ConstantStruct>(V);
- if (isa<ConstantPointerNull>(CS->getOperand(1)))
- continue;
- Function *F = cast<Function>(CS->getOperand(1));
- if (DM.GetTypeSystem().GetFunctionAnnotation(F) == nullptr)
- DM.GetTypeSystem().AddFunctionAnnotation(F);
- }
- }
- }
- void RemoveUnusedRayQuery(Module &M) {
- hlsl::OP *hlslOP = M.GetDxilModule().GetOP();
- llvm::Function *AllocFn = hlslOP->GetOpFunc(
- DXIL::OpCode::AllocateRayQuery, Type::getVoidTy(M.getContext()));
- SmallVector<CallInst*, 4> DeadInsts;
- for (auto U : AllocFn->users()) {
- if (CallInst *CI = dyn_cast<CallInst>(U)) {
- if (CI->user_empty()) {
- DeadInsts.emplace_back(CI);
- }
- }
- }
- for (auto CI : DeadInsts) {
- CI->eraseFromParent();
- }
- if (AllocFn->user_empty()) {
- AllocFn->eraseFromParent();
- }
- }
- // Convert all uses of dx.break() into per-function load/cmp of dx.break.cond global constant
- void LowerDxBreak(Module &M) {
- if (Function *BreakFunc = M.getFunction(DXIL::kDxBreakFuncName)) {
- if (!BreakFunc->use_empty()) {
- llvm::Type *i32Ty = llvm::Type::getInt32Ty(M.getContext());
- Type *i32ArrayTy = ArrayType::get(i32Ty, 1);
- unsigned int Values[1] = { 0 };
- Constant *InitialValue = ConstantDataArray::get(M.getContext(), Values);
- Constant *GV = new GlobalVariable(M, i32ArrayTy, true,
- GlobalValue::InternalLinkage,
- InitialValue, DXIL::kDxBreakCondName);
- Constant *Indices[] = { ConstantInt::get(i32Ty, 0), ConstantInt::get(i32Ty, 0) };
- Constant *Gep = ConstantExpr::getGetElementPtr(nullptr, GV, Indices);
- SmallDenseMap<llvm::Function*, llvm::ICmpInst*, 16> DxBreakCmpMap;
- // Replace all uses of dx.break with references to the constant global
- for (auto I = BreakFunc->user_begin(), E = BreakFunc->user_end(); I != E;) {
- User *U = *I++;
- CallInst *CI = cast<CallInst>(U);
- Function *F = CI->getParent()->getParent();
- ICmpInst *Cmp = DxBreakCmpMap.lookup(F);
- if (!Cmp) {
- Instruction *IP = dxilutil::FirstNonAllocaInsertionPt(F);
- LoadInst *LI = new LoadInst(Gep, nullptr, false, IP);
- Cmp = new ICmpInst(IP, ICmpInst::ICMP_EQ, LI, llvm::ConstantInt::get(i32Ty,0));
- DxBreakCmpMap[F] = Cmp;
- }
- CI->replaceAllUsesWith(Cmp);
- CI->eraseFromParent();
- }
- }
- BreakFunc->eraseFromParent();
- }
- }
- };
- }
- char DxilFinalizeModule::ID = 0;
- ModulePass *llvm::createDxilFinalizeModulePass() {
- return new DxilFinalizeModule();
- }
- INITIALIZE_PASS(DxilFinalizeModule, "hlsl-dxilfinalize", "HLSL DXIL Finalize Module", false, false)
- ///////////////////////////////////////////////////////////////////////////////
- namespace {
- typedef MapVector< PHINode*, SmallVector<Value*,8> > PHIReplacementMap;
- bool RemoveAddrSpaceCasts(Value *Val, Value *NewVal,
- PHIReplacementMap &phiReplacements,
- DenseMap<Value*, Value*> &valueMap) {
- bool bChanged = false;
- for (auto itU = Val->use_begin(), itEnd = Val->use_end(); itU != itEnd; ) {
- Use &use = *(itU++);
- User *user = use.getUser();
- Value *userReplacement = user;
- bool bConstructReplacement = false;
- bool bCleanupInst = false;
- auto valueMapIter = valueMap.find(user);
- if (valueMapIter != valueMap.end())
- userReplacement = valueMapIter->second;
- else if (Val != NewVal)
- bConstructReplacement = true;
- if (ConstantExpr* CE = dyn_cast<ConstantExpr>(user)) {
- if (CE->getOpcode() == Instruction::BitCast) {
- if (bConstructReplacement) {
- // Replicate bitcast in target address space
- Type* NewTy = PointerType::get(
- CE->getType()->getPointerElementType(),
- NewVal->getType()->getPointerAddressSpace());
- userReplacement = ConstantExpr::getBitCast(cast<Constant>(NewVal), NewTy);
- }
- } else if (CE->getOpcode() == Instruction::GetElementPtr) {
- if (bConstructReplacement) {
- // Replicate GEP in target address space
- GEPOperator *GEP = cast<GEPOperator>(CE);
- SmallVector<Value*, 8> idxList(GEP->idx_begin(), GEP->idx_end());
- userReplacement = ConstantExpr::getGetElementPtr(
- nullptr, cast<Constant>(NewVal), idxList, GEP->isInBounds());
- }
- } else if (CE->getOpcode() == Instruction::AddrSpaceCast) {
- userReplacement = NewVal;
- bConstructReplacement = false;
- } else {
- DXASSERT(false, "RemoveAddrSpaceCasts: unhandled pointer ConstantExpr");
- }
- } else if (Instruction *I = dyn_cast<Instruction>(user)) {
- if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(user)) {
- if (bConstructReplacement) {
- IRBuilder<> Builder(GEP);
- SmallVector<Value*, 8> idxList(GEP->idx_begin(), GEP->idx_end());
- if (GEP->isInBounds())
- userReplacement = Builder.CreateInBoundsGEP(NewVal, idxList, GEP->getName());
- else
- userReplacement = Builder.CreateGEP(NewVal, idxList, GEP->getName());
- }
- } else if (BitCastInst *BC = dyn_cast<BitCastInst>(user)) {
- if (bConstructReplacement) {
- IRBuilder<> Builder(BC);
- Type* NewTy = PointerType::get(
- BC->getType()->getPointerElementType(),
- NewVal->getType()->getPointerAddressSpace());
- userReplacement = Builder.CreateBitCast(NewVal, NewTy);
- }
- } else if (PHINode *PHI = dyn_cast<PHINode>(user)) {
- // set replacement phi values for PHI pass
- unsigned numValues = PHI->getNumIncomingValues();
- auto &phiValues = phiReplacements[PHI];
- if (phiValues.empty())
- phiValues.resize(numValues, nullptr);
- for (unsigned idx = 0; idx < numValues; ++idx) {
- if (phiValues[idx] == nullptr &&
- PHI->getIncomingValue(idx) == Val) {
- phiValues[idx] = NewVal;
- bChanged = true;
- }
- }
- continue;
- } else if (isa<AddrSpaceCastInst>(user)) {
- userReplacement = NewVal;
- bConstructReplacement = false;
- bCleanupInst = true;
- } else if (isa<CallInst>(user)) {
- continue;
- } else {
- if (Val != NewVal) {
- use.set(NewVal);
- bChanged = true;
- }
- continue;
- }
- }
- if (bConstructReplacement && user != userReplacement)
- valueMap[user] = userReplacement;
- bChanged |= RemoveAddrSpaceCasts(user, userReplacement, phiReplacements,
- valueMap);
- if (bCleanupInst && user->use_empty()) {
- // Clean up old instruction if it's now unused.
- // Safe during this use iteration when only one use of V in instruction.
- if (Instruction *I = dyn_cast<Instruction>(user))
- I->eraseFromParent();
- bChanged = true;
- }
- }
- return bChanged;
- }
- }
- bool CleanupSharedMemoryAddrSpaceCast(Module &M) {
- bool bChanged = false;
- // Eliminate address space casts if possible
- // Collect phi nodes so we can replace iteratively after pass over GVs
- PHIReplacementMap phiReplacements;
- DenseMap<Value*, Value*> valueMap;
- for (GlobalVariable &GV : M.globals()) {
- if (dxilutil::IsSharedMemoryGlobal(&GV)) {
- bChanged |= RemoveAddrSpaceCasts(&GV, &GV, phiReplacements,
- valueMap);
- }
- }
- bool bConverged = false;
- while (!phiReplacements.empty() && !bConverged) {
- bConverged = true;
- for (auto &phiReplacement : phiReplacements) {
- PHINode *PHI = phiReplacement.first;
- unsigned origAddrSpace = PHI->getType()->getPointerAddressSpace();
- unsigned incomingAddrSpace = UINT_MAX;
- bool bReplacePHI = true;
- bool bRemovePHI = false;
- for (auto V : phiReplacement.second) {
- if (nullptr == V) {
- // cannot replace phi (yet)
- bReplacePHI = false;
- break;
- }
- unsigned addrSpace = V->getType()->getPointerAddressSpace();
- if (incomingAddrSpace == UINT_MAX) {
- incomingAddrSpace = addrSpace;
- } else if (addrSpace != incomingAddrSpace) {
- bRemovePHI = true;
- break;
- }
- }
- if (origAddrSpace == incomingAddrSpace)
- bRemovePHI = true;
- if (bRemovePHI) {
- // Cannot replace phi. Remove it and restart.
- phiReplacements.erase(PHI);
- bConverged = false;
- break;
- }
- if (!bReplacePHI)
- continue;
- auto &NewVal = valueMap[PHI];
- PHINode *NewPHI = nullptr;
- if (NewVal) {
- NewPHI = cast<PHINode>(NewVal);
- } else {
- IRBuilder<> Builder(PHI);
- NewPHI = Builder.CreatePHI(
- PointerType::get(PHI->getType()->getPointerElementType(),
- incomingAddrSpace),
- PHI->getNumIncomingValues(),
- PHI->getName());
- NewVal = NewPHI;
- for (unsigned idx = 0; idx < PHI->getNumIncomingValues(); idx++) {
- NewPHI->addIncoming(phiReplacement.second[idx],
- PHI->getIncomingBlock(idx));
- }
- }
- if (RemoveAddrSpaceCasts(PHI, NewPHI, phiReplacements,
- valueMap)) {
- bConverged = false;
- bChanged = true;
- break;
- }
- if (PHI->use_empty()) {
- phiReplacements.erase(PHI);
- bConverged = false;
- bChanged = true;
- break;
- }
- }
- }
- // Cleanup unused replacement instructions
- SmallVector<WeakVH, 8> cleanupInsts;
- for (auto it : valueMap) {
- if (isa<Instruction>(it.first))
- cleanupInsts.push_back(it.first);
- if (isa<Instruction>(it.second))
- cleanupInsts.push_back(it.second);
- }
- for (auto V : cleanupInsts) {
- if (!V)
- continue;
- if (PHINode *PHI = dyn_cast<PHINode>(V))
- RecursivelyDeleteDeadPHINode(PHI);
- else if (Instruction *I = dyn_cast<Instruction>(V))
- RecursivelyDeleteTriviallyDeadInstructions(I);
- }
- return bChanged;
- }
- class DxilCleanupAddrSpaceCast : public ModulePass {
- public:
- static char ID; // Pass identification, replacement for typeid
- explicit DxilCleanupAddrSpaceCast() : ModulePass(ID) {}
- const char *getPassName() const override { return "HLSL DXIL Cleanup Address Space Cast"; }
- bool runOnModule(Module &M) override {
- return CleanupSharedMemoryAddrSpaceCast(M);
- }
- };
- char DxilCleanupAddrSpaceCast::ID = 0;
- ModulePass *llvm::createDxilCleanupAddrSpaceCastPass() {
- return new DxilCleanupAddrSpaceCast();
- }
- INITIALIZE_PASS(DxilCleanupAddrSpaceCast, "hlsl-dxil-cleanup-addrspacecast", "HLSL DXIL Cleanup Address Space Cast", false, false)
- ///////////////////////////////////////////////////////////////////////////////
- namespace {
- class DxilEmitMetadata : public ModulePass {
- public:
- static char ID; // Pass identification, replacement for typeid
- explicit DxilEmitMetadata() : ModulePass(ID) {}
- const char *getPassName() const override { return "HLSL DXIL Metadata Emit"; }
- bool runOnModule(Module &M) override {
- if (M.HasDxilModule()) {
- DxilModule::ClearDxilMetadata(M);
- patchIsFrontfaceTy(M);
- M.GetDxilModule().EmitDxilMetadata();
- return true;
- }
- return false;
- }
- private:
- void patchIsFrontfaceTy(Module &M);
- };
- void patchIsFrontface(DxilSignatureElement &Elt, bool bForceUint) {
- // If force to uint, change i1 to u32.
- // If not force to uint, change u32 to i1.
- if (bForceUint && Elt.GetCompType() == CompType::Kind::I1)
- Elt.SetCompType(CompType::Kind::U32);
- else if (!bForceUint && Elt.GetCompType() == CompType::Kind::U32)
- Elt.SetCompType(CompType::Kind::I1);
- }
- void patchIsFrontface(DxilSignature &sig, bool bForceUint) {
- for (auto &Elt : sig.GetElements()) {
- if (Elt->GetSemantic()->GetKind() == Semantic::Kind::IsFrontFace) {
- patchIsFrontface(*Elt, bForceUint);
- }
- }
- }
- void DxilEmitMetadata::patchIsFrontfaceTy(Module &M) {
- DxilModule &DM = M.GetDxilModule();
- const ShaderModel *pSM = DM.GetShaderModel();
- if (!pSM->IsGS() && !pSM->IsPS())
- return;
- unsigned ValMajor, ValMinor;
- DM.GetValidatorVersion(ValMajor, ValMinor);
- bool bForceUint = ValMajor == 0 || (ValMajor >= 1 && ValMinor >= 2);
- if (pSM->IsPS()) {
- patchIsFrontface(DM.GetInputSignature(), bForceUint);
- } else if (pSM->IsGS()) {
- patchIsFrontface(DM.GetOutputSignature(), bForceUint);
- }
- }
- }
- char DxilEmitMetadata::ID = 0;
- ModulePass *llvm::createDxilEmitMetadataPass() {
- return new DxilEmitMetadata();
- }
- INITIALIZE_PASS(DxilEmitMetadata, "hlsl-dxilemit", "HLSL DXIL Metadata Emit", false, false)
- ///////////////////////////////////////////////////////////////////////////////
- namespace {
- const StringRef UniNoWaveSensitiveGradientErrMsg =
- "Gradient operations are not affected by wave-sensitive data or control "
- "flow.";
- class DxilValidateWaveSensitivity : public ModulePass {
- public:
- static char ID; // Pass identification, replacement for typeid
- explicit DxilValidateWaveSensitivity() : ModulePass(ID) {}
- const char *getPassName() const override {
- return "HLSL DXIL wave sensitiveity validation";
- }
- bool runOnModule(Module &M) override {
- // Only check ps and lib profile.
- DxilModule &DM = M.GetDxilModule();
- const ShaderModel *pSM = DM.GetShaderModel();
- if (!pSM->IsPS() && !pSM->IsLib())
- return false;
- SmallVector<CallInst *, 16> gradientOps;
- SmallVector<CallInst *, 16> barriers;
- SmallVector<CallInst *, 16> waveOps;
- for (auto &F : M) {
- if (!F.isDeclaration())
- continue;
- for (User *U : F.users()) {
- CallInst *CI = dyn_cast<CallInst>(U);
- if (!CI)
- continue;
- Function *FCalled = CI->getCalledFunction();
- if (!FCalled || !FCalled->isDeclaration())
- continue;
- if (!hlsl::OP::IsDxilOpFunc(FCalled))
- continue;
- DXIL::OpCode dxilOpcode = hlsl::OP::GetDxilOpFuncCallInst(CI);
- if (OP::IsDxilOpWave(dxilOpcode)) {
- waveOps.emplace_back(CI);
- }
- if (OP::IsDxilOpGradient(dxilOpcode)) {
- gradientOps.push_back(CI);
- }
- if (dxilOpcode == DXIL::OpCode::Barrier) {
- barriers.push_back(CI);
- }
- }
- }
- // Skip if not have wave op.
- if (waveOps.empty())
- return false;
- // Skip if no gradient op.
- if (gradientOps.empty())
- return false;
- for (auto &F : M) {
- if (F.isDeclaration())
- continue;
- SmallVector<CallInst *, 16> localGradientOps;
- for (CallInst *CI : gradientOps) {
- if (CI->getParent()->getParent() == &F)
- localGradientOps.emplace_back(CI);
- }
- if (localGradientOps.empty())
- continue;
- PostDominatorTree PDT;
- PDT.runOnFunction(F);
- std::unique_ptr<WaveSensitivityAnalysis> WaveVal(
- WaveSensitivityAnalysis::create(PDT));
- WaveVal->Analyze(&F);
- for (CallInst *op : localGradientOps) {
- if (WaveVal->IsWaveSensitive(op)) {
- dxilutil::EmitWarningOnInstruction(op,
- UniNoWaveSensitiveGradientErrMsg);
- }
- }
- }
- return false;
- }
- };
- }
- char DxilValidateWaveSensitivity::ID = 0;
- ModulePass *llvm::createDxilValidateWaveSensitivityPass() {
- return new DxilValidateWaveSensitivity();
- }
- INITIALIZE_PASS(DxilValidateWaveSensitivity, "hlsl-validate-wave-sensitivity", "HLSL DXIL wave sensitiveity validation", false, false)
- namespace {
- // Cull blocks from BreakBBs that containing instructions that are sensitive to the wave-sensitive Inst
- // Sensitivity entails being an eventual user of the Inst and also belonging to a block with
- // a break conditional on dx.break that breaks out of a loop that contains WaveCI
- // LInfo is needed to determine loop contents. VisitedPhis is needed to prevent infinit looping.
- static void CullSensitiveBlocks(LoopInfo *LInfo, BasicBlock *WaveBB, BasicBlock *LastBB, Instruction *Inst,
- SmallPtrSet<Instruction *, 16> &VisitedPhis,
- SmallDenseMap<BasicBlock *, Instruction *, 16> &BreakBBs) {
- BasicBlock *BB = Inst->getParent();
- Loop *BreakLoop = LInfo->getLoopFor(BB);
- // If this instruction isn't in a loop, there is no need to track its sensitivity further
- if (!BreakLoop || BreakBBs.empty())
- return;
- // To prevent infinite looping, only visit each PHI once
- // If we've seen this PHI before, don't reprocess it
- if (isa<PHINode>(Inst) && !VisitedPhis.insert(Inst).second)
- return;
- // If this BB wasn't already just processed, handle it now
- if (LastBB != BB) {
- // Determine if the instruction's block has an artificially-conditional break
- // and breaks out of a loop that contains the waveCI
- BranchInst *BI = dyn_cast<BranchInst>(BB->getTerminator());
- if (BI && BI->isConditional() && BreakLoop->contains(WaveBB))
- BreakBBs.erase(BB);
- }
- // Recurse on the users
- for (User *U : Inst->users()) {
- Instruction *I = cast<Instruction>(U);
- CullSensitiveBlocks(LInfo, WaveBB, BB, I, VisitedPhis, BreakBBs);
- }
- }
- // Collect blocks that end in a dx.break dependent branch by tracing the descendants of BreakFunc
- // that are found in ThisFunc and store the block and call instruction in BreakBBs
- static void CollectBreakBlocks(Function *BreakFunc, Function *ThisFunc,
- SmallDenseMap<BasicBlock *, Instruction *, 16> &BreakBBs) {
- for (User *U : BreakFunc->users()) {
- SmallVector<User *, 16> WorkList;
- Instruction *CI = cast<Instruction>(U);
- // If this user doesn't pertain to the current function, skip it.
- if (CI->getParent()->getParent() != ThisFunc)
- continue;
- WorkList.append(CI->user_begin(), CI->user_end());
- while (!WorkList.empty()) {
- Instruction *I = dyn_cast<Instruction>(WorkList.pop_back_val());
- // When we find a Branch that depends on dx.break, save it and stop
- // This should almost always be the first user of the Call Inst
- // If not, iterate on the users
- if (BranchInst *BI = dyn_cast<BranchInst>(I))
- BreakBBs[BI->getParent()] = CI;
- else
- WorkList.append(I->user_begin(), I->user_end());
- }
- }
- }
- // A pass to remove conditions from breaks that do not contain instructions that
- // depend on wave operations that are in the loop that the break leaves.
- class CleanupDxBreak : public FunctionPass {
- public:
- static char ID; // Pass identification, replacement for typeid
- explicit CleanupDxBreak() : FunctionPass(ID) {}
- const char *getPassName() const override { return "HLSL Remove unnecessary dx.break conditions"; }
- void getAnalysisUsage(AnalysisUsage &AU) const override {
- AU.addRequired<LoopInfoWrapperPass>();
- }
- LoopInfo *LInfo;
- bool runOnFunction(Function &F) override {
- if (F.isDeclaration())
- return false;
- Module *M = F.getEntryBlock().getModule();
- Function *BreakFunc = M->getFunction(DXIL::kDxBreakFuncName);
- if (!BreakFunc)
- return false;
- LInfo = &getAnalysis<LoopInfoWrapperPass>().getLoopInfo();
- // Collect the blocks that depend on dx.break and the instructions that call dx.break()
- SmallDenseMap<BasicBlock *, Instruction *, 16> BreakBBs;
- CollectBreakBlocks(BreakFunc, &F, BreakBBs);
- if (BreakBBs.empty())
- return false;
- // For each wave operation, remove all the dx.break blocks that are sensitive to it
- for (Function &IF : M->functions()) {
- HLOpcodeGroup opgroup = hlsl::GetHLOpcodeGroup(&IF);
- // Only consider wave-sensitive intrinsics or extintrinsics
- if (IF.isDeclaration() && IsHLWaveSensitive(&IF) && !BreakBBs.empty() &&
- (opgroup == HLOpcodeGroup::HLIntrinsic || opgroup == HLOpcodeGroup::HLExtIntrinsic)) {
- // For each user of the function, trace all its users to remove the blocks
- for (User *U : IF.users()) {
- CallInst *CI = cast<CallInst>(U);
- if (CI->getParent()->getParent() == &F) {
- SmallPtrSet<Instruction *, 16> VisitedPhis;
- CullSensitiveBlocks(LInfo, CI->getParent(), nullptr, CI, VisitedPhis, BreakBBs);
- }
- }
- }
- }
- bool Changed = false;
- // Revert artificially conditional breaks in non-wave-sensitive blocks that remain in BreakBBs
- Constant *C = ConstantInt::get(Type::getInt1Ty(M->getContext()), 1);
- for (auto &BB : BreakBBs) {
- // Replace the call instruction with a constant boolen
- BB.second->replaceAllUsesWith(C);
- BB.second->eraseFromParent();
- Changed = true;
- }
- return Changed;
- }
- };
- }
- char CleanupDxBreak::ID = 0;
- INITIALIZE_PASS_BEGIN(CleanupDxBreak, "hlsl-cleanup-dxbreak", "HLSL Remove unnecessary dx.break conditions", false, false)
- INITIALIZE_PASS_DEPENDENCY(LoopInfoWrapperPass)
- INITIALIZE_PASS_END(CleanupDxBreak, "hlsl-cleanup-dxbreak", "HLSL Remove unnecessary dx.break conditions", false, false)
- FunctionPass *llvm::createCleanupDxBreakPass() {
- return new CleanupDxBreak();
- }
|