1900b6335SEugene Zelenko //===- ImplicitNullChecks.cpp - Fold null checks into memory accesses -----===// 269fad079SSanjoy Das // 32946cd70SChandler Carruth // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. 42946cd70SChandler Carruth // See https://llvm.org/LICENSE.txt for license information. 52946cd70SChandler Carruth // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception 669fad079SSanjoy Das // 769fad079SSanjoy Das //===----------------------------------------------------------------------===// 869fad079SSanjoy Das // 969fad079SSanjoy Das // This pass turns explicit null checks of the form 1069fad079SSanjoy Das // 1169fad079SSanjoy Das // test %r10, %r10 1269fad079SSanjoy Das // je throw_npe 1369fad079SSanjoy Das // movl (%r10), %esi 1469fad079SSanjoy Das // ... 1569fad079SSanjoy Das // 1669fad079SSanjoy Das // to 1769fad079SSanjoy Das // 1869fad079SSanjoy Das // faulting_load_op("movl (%r10), %esi", throw_npe) 1969fad079SSanjoy Das // ... 2069fad079SSanjoy Das // 2169fad079SSanjoy Das // With the help of a runtime that understands the .fault_maps section, 2269fad079SSanjoy Das // faulting_load_op branches to throw_npe if executing movl (%r10), %esi incurs 2369fad079SSanjoy Das // a page fault. 2451c220cbSSerguei Katkov // Store and LoadStore are also supported. 2569fad079SSanjoy Das // 2669fad079SSanjoy Das //===----------------------------------------------------------------------===// 2769fad079SSanjoy Das 28900b6335SEugene Zelenko #include "llvm/ADT/ArrayRef.h" 29900b6335SEugene Zelenko #include "llvm/ADT/None.h" 30900b6335SEugene Zelenko #include "llvm/ADT/Optional.h" 31900b6335SEugene Zelenko #include "llvm/ADT/STLExtras.h" 3269fad079SSanjoy Das #include "llvm/ADT/SmallVector.h" 338ee6a30bSSanjoy Das #include "llvm/ADT/Statistic.h" 34e57bf680SSanjoy Das #include "llvm/Analysis/AliasAnalysis.h" 35900b6335SEugene Zelenko #include "llvm/Analysis/MemoryLocation.h" 362f63cbccSSanjoy Das #include "llvm/CodeGen/FaultMaps.h" 37900b6335SEugene Zelenko #include "llvm/CodeGen/MachineBasicBlock.h" 3869fad079SSanjoy Das #include "llvm/CodeGen/MachineFunction.h" 3969fad079SSanjoy Das #include "llvm/CodeGen/MachineFunctionPass.h" 40900b6335SEugene Zelenko #include "llvm/CodeGen/MachineInstr.h" 4169fad079SSanjoy Das #include "llvm/CodeGen/MachineInstrBuilder.h" 426bda14b3SChandler Carruth #include "llvm/CodeGen/MachineMemOperand.h" 436bda14b3SChandler Carruth #include "llvm/CodeGen/MachineOperand.h" 446bda14b3SChandler Carruth #include "llvm/CodeGen/MachineRegisterInfo.h" 45900b6335SEugene Zelenko #include "llvm/CodeGen/PseudoSourceValue.h" 463f833edcSDavid Blaikie #include "llvm/CodeGen/TargetInstrInfo.h" 47b3bde2eaSDavid Blaikie #include "llvm/CodeGen/TargetOpcodes.h" 48b3bde2eaSDavid Blaikie #include "llvm/CodeGen/TargetRegisterInfo.h" 49b3bde2eaSDavid Blaikie #include "llvm/CodeGen/TargetSubtargetInfo.h" 5069fad079SSanjoy Das #include "llvm/IR/BasicBlock.h" 51900b6335SEugene Zelenko #include "llvm/IR/DebugLoc.h" 5200038784SChen Li #include "llvm/IR/LLVMContext.h" 5305da2fe5SReid Kleckner #include "llvm/InitializePasses.h" 54900b6335SEugene Zelenko #include "llvm/MC/MCInstrDesc.h" 55900b6335SEugene Zelenko #include "llvm/MC/MCRegisterInfo.h" 56900b6335SEugene Zelenko #include "llvm/Pass.h" 5769fad079SSanjoy Das #include "llvm/Support/CommandLine.h" 58900b6335SEugene Zelenko #include <cassert> 59900b6335SEugene Zelenko #include <cstdint> 60900b6335SEugene Zelenko #include <iterator> 6169fad079SSanjoy Das 6269fad079SSanjoy Das using namespace llvm; 6369fad079SSanjoy Das 64c27a18f3SChad Rosier static cl::opt<int> PageSize("imp-null-check-page-size", 65c27a18f3SChad Rosier cl::desc("The page size of the target in bytes"), 668065f0b9SZachary Turner cl::init(4096), cl::Hidden); 6769fad079SSanjoy Das 689a129807SSanjoy Das static cl::opt<unsigned> MaxInstsToConsider( 699a129807SSanjoy Das "imp-null-max-insts-to-consider", 709a129807SSanjoy Das cl::desc("The max number of instructions to consider hoisting loads over " 719a129807SSanjoy Das "(the algorithm is quadratic over this number)"), 728065f0b9SZachary Turner cl::Hidden, cl::init(8)); 739a129807SSanjoy Das 748ee6a30bSSanjoy Das #define DEBUG_TYPE "implicit-null-checks" 758ee6a30bSSanjoy Das 768ee6a30bSSanjoy Das STATISTIC(NumImplicitNullChecks, 778ee6a30bSSanjoy Das "Number of explicit null checks made implicit"); 788ee6a30bSSanjoy Das 7969fad079SSanjoy Das namespace { 8069fad079SSanjoy Das 8169fad079SSanjoy Das class ImplicitNullChecks : public MachineFunctionPass { 829a129807SSanjoy Das /// Return true if \c computeDependence can process \p MI. 839a129807SSanjoy Das static bool canHandle(const MachineInstr *MI); 849a129807SSanjoy Das 859a129807SSanjoy Das /// Helper function for \c computeDependence. Return true if \p A 869a129807SSanjoy Das /// and \p B do not have any dependences between them, and can be 879a129807SSanjoy Das /// re-ordered without changing program semantics. 889a129807SSanjoy Das bool canReorder(const MachineInstr *A, const MachineInstr *B); 899a129807SSanjoy Das 909a129807SSanjoy Das /// A data type for representing the result computed by \c 919a129807SSanjoy Das /// computeDependence. States whether it is okay to reorder the 929a129807SSanjoy Das /// instruction passed to \c computeDependence with at most one 9358963e43SFangrui Song /// dependency. 949a129807SSanjoy Das struct DependenceResult { 959a129807SSanjoy Das /// Can we actually re-order \p MI with \p Insts (see \c 969a129807SSanjoy Das /// computeDependence). 979a129807SSanjoy Das bool CanReorder; 989a129807SSanjoy Das 999a129807SSanjoy Das /// If non-None, then an instruction in \p Insts that also must be 1009a129807SSanjoy Das /// hoisted. 1019a129807SSanjoy Das Optional<ArrayRef<MachineInstr *>::iterator> PotentialDependence; 1029a129807SSanjoy Das 1039a129807SSanjoy Das /*implicit*/ DependenceResult( 1049a129807SSanjoy Das bool CanReorder, 1059a129807SSanjoy Das Optional<ArrayRef<MachineInstr *>::iterator> PotentialDependence) 1069a129807SSanjoy Das : CanReorder(CanReorder), PotentialDependence(PotentialDependence) { 1079a129807SSanjoy Das assert((!PotentialDependence || CanReorder) && 1089a129807SSanjoy Das "!CanReorder && PotentialDependence.hasValue() not allowed!"); 1099a129807SSanjoy Das } 1109a129807SSanjoy Das }; 1119a129807SSanjoy Das 1129a129807SSanjoy Das /// Compute a result for the following question: can \p MI be 1139a129807SSanjoy Das /// re-ordered from after \p Insts to before it. 1149a129807SSanjoy Das /// 1159a129807SSanjoy Das /// \c canHandle should return true for all instructions in \p 1169a129807SSanjoy Das /// Insts. 1179a129807SSanjoy Das DependenceResult computeDependence(const MachineInstr *MI, 118cb0bab86SFangrui Song ArrayRef<MachineInstr *> Block); 1199a129807SSanjoy Das 12069fad079SSanjoy Das /// Represents one null check that can be made implicit. 121e173b9aeSSanjoy Das class NullCheck { 12269fad079SSanjoy Das // The memory operation the null check can be folded into. 12369fad079SSanjoy Das MachineInstr *MemOperation; 12469fad079SSanjoy Das 12569fad079SSanjoy Das // The instruction actually doing the null check (Ptr != 0). 12669fad079SSanjoy Das MachineInstr *CheckOperation; 12769fad079SSanjoy Das 12869fad079SSanjoy Das // The block the check resides in. 12969fad079SSanjoy Das MachineBasicBlock *CheckBlock; 13069fad079SSanjoy Das 131572e03a3SEric Christopher // The block branched to if the pointer is non-null. 13269fad079SSanjoy Das MachineBasicBlock *NotNullSucc; 13369fad079SSanjoy Das 134572e03a3SEric Christopher // The block branched to if the pointer is null. 13569fad079SSanjoy Das MachineBasicBlock *NullSucc; 13669fad079SSanjoy Das 1370909ca13SHiroshi Inoue // If this is non-null, then MemOperation has a dependency on this 138e57bf680SSanjoy Das // instruction; and it needs to be hoisted to execute before MemOperation. 139e57bf680SSanjoy Das MachineInstr *OnlyDependency; 140e57bf680SSanjoy Das 141e173b9aeSSanjoy Das public: 14269fad079SSanjoy Das explicit NullCheck(MachineInstr *memOperation, MachineInstr *checkOperation, 14369fad079SSanjoy Das MachineBasicBlock *checkBlock, 14469fad079SSanjoy Das MachineBasicBlock *notNullSucc, 145e57bf680SSanjoy Das MachineBasicBlock *nullSucc, 146e57bf680SSanjoy Das MachineInstr *onlyDependency) 14769fad079SSanjoy Das : MemOperation(memOperation), CheckOperation(checkOperation), 148e57bf680SSanjoy Das CheckBlock(checkBlock), NotNullSucc(notNullSucc), NullSucc(nullSucc), 149e57bf680SSanjoy Das OnlyDependency(onlyDependency) {} 150e173b9aeSSanjoy Das 151e173b9aeSSanjoy Das MachineInstr *getMemOperation() const { return MemOperation; } 152e173b9aeSSanjoy Das 153e173b9aeSSanjoy Das MachineInstr *getCheckOperation() const { return CheckOperation; } 154e173b9aeSSanjoy Das 155e173b9aeSSanjoy Das MachineBasicBlock *getCheckBlock() const { return CheckBlock; } 156e173b9aeSSanjoy Das 157e173b9aeSSanjoy Das MachineBasicBlock *getNotNullSucc() const { return NotNullSucc; } 158e173b9aeSSanjoy Das 159e173b9aeSSanjoy Das MachineBasicBlock *getNullSucc() const { return NullSucc; } 160e57bf680SSanjoy Das 161e57bf680SSanjoy Das MachineInstr *getOnlyDependency() const { return OnlyDependency; } 16269fad079SSanjoy Das }; 16369fad079SSanjoy Das 16469fad079SSanjoy Das const TargetInstrInfo *TII = nullptr; 16569fad079SSanjoy Das const TargetRegisterInfo *TRI = nullptr; 166e57bf680SSanjoy Das AliasAnalysis *AA = nullptr; 167eef785c1SSanjoy Das MachineFrameInfo *MFI = nullptr; 16869fad079SSanjoy Das 16969fad079SSanjoy Das bool analyzeBlockForNullChecks(MachineBasicBlock &MBB, 17069fad079SSanjoy Das SmallVectorImpl<NullCheck> &NullCheckList); 1712f63cbccSSanjoy Das MachineInstr *insertFaultingInstr(MachineInstr *MI, MachineBasicBlock *MBB, 1724e1d389aSQuentin Colombet MachineBasicBlock *HandlerMBB); 17369fad079SSanjoy Das void rewriteNullChecks(ArrayRef<NullCheck> NullCheckList); 17469fad079SSanjoy Das 175eef785c1SSanjoy Das enum AliasResult { 176eef785c1SSanjoy Das AR_NoAlias, 177eef785c1SSanjoy Das AR_MayAlias, 178eef785c1SSanjoy Das AR_WillAliasEverything 179eef785c1SSanjoy Das }; 180900b6335SEugene Zelenko 181eef785c1SSanjoy Das /// Returns AR_NoAlias if \p MI memory operation does not alias with 182eef785c1SSanjoy Das /// \p PrevMI, AR_MayAlias if they may alias and AR_WillAliasEverything if 183eef785c1SSanjoy Das /// they may alias and any further memory operation may alias with \p PrevMI. 184238c9d63SBjorn Pettersson AliasResult areMemoryOpsAliased(const MachineInstr &MI, 185238c9d63SBjorn Pettersson const MachineInstr *PrevMI) const; 18615e50b51SSanjoy Das 187eef785c1SSanjoy Das enum SuitabilityResult { 188eef785c1SSanjoy Das SR_Suitable, 189eef785c1SSanjoy Das SR_Unsuitable, 190eef785c1SSanjoy Das SR_Impossible 191eef785c1SSanjoy Das }; 192900b6335SEugene Zelenko 19315e50b51SSanjoy Das /// Return SR_Suitable if \p MI a memory operation that can be used to 19415e50b51SSanjoy Das /// implicitly null check the value in \p PointerReg, SR_Unsuitable if 19515e50b51SSanjoy Das /// \p MI cannot be used to null check and SR_Impossible if there is 19615e50b51SSanjoy Das /// no sense to continue lookup due to any other instruction will not be able 19715e50b51SSanjoy Das /// to be used. \p PrevInsts is the set of instruction seen since 198eef785c1SSanjoy Das /// the explicit null check on \p PointerReg. 199238c9d63SBjorn Pettersson SuitabilityResult isSuitableMemoryOp(const MachineInstr &MI, 200238c9d63SBjorn Pettersson unsigned PointerReg, 201eef785c1SSanjoy Das ArrayRef<MachineInstr *> PrevInsts); 20250fef432SSanjoy Das 203425573a2SAnna Thomas /// Returns true if \p DependenceMI can clobber the liveIns in NullSucc block 204425573a2SAnna Thomas /// if it was hoisted to the NullCheck block. This is used by caller 205425573a2SAnna Thomas /// canHoistInst to decide if DependenceMI can be hoisted safely. 206425573a2SAnna Thomas bool canDependenceHoistingClobberLiveIns(MachineInstr *DependenceMI, 207b1b98063SAnna Thomas MachineBasicBlock *NullSucc); 208425573a2SAnna Thomas 2098f976ba0SHiroshi Inoue /// Return true if \p FaultingMI can be hoisted from after the 21050fef432SSanjoy Das /// instructions in \p InstsSeenSoFar to before them. Set \p Dependence to a 2116a0ed57aSSimon Pilgrim /// non-null value if we also need to (and legally can) hoist a dependency. 212b1b98063SAnna Thomas bool canHoistInst(MachineInstr *FaultingMI, 21350fef432SSanjoy Das ArrayRef<MachineInstr *> InstsSeenSoFar, 21450fef432SSanjoy Das MachineBasicBlock *NullSucc, MachineInstr *&Dependence); 21550fef432SSanjoy Das 21669fad079SSanjoy Das public: 21769fad079SSanjoy Das static char ID; 21869fad079SSanjoy Das 21969fad079SSanjoy Das ImplicitNullChecks() : MachineFunctionPass(ID) { 22069fad079SSanjoy Das initializeImplicitNullChecksPass(*PassRegistry::getPassRegistry()); 22169fad079SSanjoy Das } 22269fad079SSanjoy Das 22369fad079SSanjoy Das bool runOnMachineFunction(MachineFunction &MF) override; 224900b6335SEugene Zelenko 225e57bf680SSanjoy Das void getAnalysisUsage(AnalysisUsage &AU) const override { 226e57bf680SSanjoy Das AU.addRequired<AAResultsWrapperPass>(); 227e57bf680SSanjoy Das MachineFunctionPass::getAnalysisUsage(AU); 228e57bf680SSanjoy Das } 229ad154c83SDerek Schuff 230ad154c83SDerek Schuff MachineFunctionProperties getRequiredProperties() const override { 231ad154c83SDerek Schuff return MachineFunctionProperties().set( 2321eb47368SMatthias Braun MachineFunctionProperties::Property::NoVRegs); 233ad154c83SDerek Schuff } 23469fad079SSanjoy Das }; 235edc394f1SSanjoy Das 236900b6335SEugene Zelenko } // end anonymous namespace 237e57bf680SSanjoy Das 2389a129807SSanjoy Das bool ImplicitNullChecks::canHandle(const MachineInstr *MI) { 2396c5d5ce5SUlrich Weigand if (MI->isCall() || MI->mayRaiseFPException() || 2406c5d5ce5SUlrich Weigand MI->hasUnmodeledSideEffects()) 2419a129807SSanjoy Das return false; 2429a129807SSanjoy Das auto IsRegMask = [](const MachineOperand &MO) { return MO.isRegMask(); }; 2439a129807SSanjoy Das (void)IsRegMask; 244edc394f1SSanjoy Das 2459a129807SSanjoy Das assert(!llvm::any_of(MI->operands(), IsRegMask) && 2469a129807SSanjoy Das "Calls were filtered out above!"); 247edc394f1SSanjoy Das 24821a50ccfSPhilip Reames auto IsUnordered = [](MachineMemOperand *MMO) { return MMO->isUnordered(); }; 24921a50ccfSPhilip Reames return llvm::all_of(MI->memoperands(), IsUnordered); 250edc394f1SSanjoy Das } 251edc394f1SSanjoy Das 2529a129807SSanjoy Das ImplicitNullChecks::DependenceResult 2539a129807SSanjoy Das ImplicitNullChecks::computeDependence(const MachineInstr *MI, 2549a129807SSanjoy Das ArrayRef<MachineInstr *> Block) { 2559a129807SSanjoy Das assert(llvm::all_of(Block, canHandle) && "Check this first!"); 256900b6335SEugene Zelenko assert(!is_contained(Block, MI) && "Block must be exclusive of MI!"); 257edc394f1SSanjoy Das 2589a129807SSanjoy Das Optional<ArrayRef<MachineInstr *>::iterator> Dep; 259edc394f1SSanjoy Das 2609a129807SSanjoy Das for (auto I = Block.begin(), E = Block.end(); I != E; ++I) { 2619a129807SSanjoy Das if (canReorder(*I, MI)) 262edc394f1SSanjoy Das continue; 263edc394f1SSanjoy Das 2649a129807SSanjoy Das if (Dep == None) { 2659a129807SSanjoy Das // Found one possible dependency, keep track of it. 2669a129807SSanjoy Das Dep = I; 2679a129807SSanjoy Das } else { 2689a129807SSanjoy Das // We found two dependencies, so bail out. 2699a129807SSanjoy Das return {false, None}; 270edc394f1SSanjoy Das } 271edc394f1SSanjoy Das } 272edc394f1SSanjoy Das 2739a129807SSanjoy Das return {true, Dep}; 2749a129807SSanjoy Das } 275edc394f1SSanjoy Das 2769a129807SSanjoy Das bool ImplicitNullChecks::canReorder(const MachineInstr *A, 2779a129807SSanjoy Das const MachineInstr *B) { 2789a129807SSanjoy Das assert(canHandle(A) && canHandle(B) && "Precondition!"); 279edc394f1SSanjoy Das 2809a129807SSanjoy Das // canHandle makes sure that we _can_ correctly analyze the dependencies 2819a129807SSanjoy Das // between A and B here -- for instance, we should not be dealing with heap 2829a129807SSanjoy Das // load-store dependencies here. 2839a129807SSanjoy Das 2846a0ed57aSSimon Pilgrim for (const auto &MOA : A->operands()) { 2859a129807SSanjoy Das if (!(MOA.isReg() && MOA.getReg())) 286e57bf680SSanjoy Das continue; 287e57bf680SSanjoy Das 2880c476111SDaniel Sanders Register RegA = MOA.getReg(); 2896a0ed57aSSimon Pilgrim for (const auto &MOB : B->operands()) { 2909a129807SSanjoy Das if (!(MOB.isReg() && MOB.getReg())) 291e57bf680SSanjoy Das continue; 2929a129807SSanjoy Das 2930c476111SDaniel Sanders Register RegB = MOB.getReg(); 2949a129807SSanjoy Das 29508da2e28SSanjoy Das if (TRI->regsOverlap(RegA, RegB) && (MOA.isDef() || MOB.isDef())) 296e57bf680SSanjoy Das return false; 297e57bf680SSanjoy Das } 298edc394f1SSanjoy Das } 299edc394f1SSanjoy Das 300edc394f1SSanjoy Das return true; 301f00654e3SAlexander Kornienko } 30269fad079SSanjoy Das 30369fad079SSanjoy Das bool ImplicitNullChecks::runOnMachineFunction(MachineFunction &MF) { 30469fad079SSanjoy Das TII = MF.getSubtarget().getInstrInfo(); 30569fad079SSanjoy Das TRI = MF.getRegInfo().getTargetRegisterInfo(); 306eef785c1SSanjoy Das MFI = &MF.getFrameInfo(); 307e57bf680SSanjoy Das AA = &getAnalysis<AAResultsWrapperPass>().getAAResults(); 30869fad079SSanjoy Das 30969fad079SSanjoy Das SmallVector<NullCheck, 16> NullCheckList; 31069fad079SSanjoy Das 31169fad079SSanjoy Das for (auto &MBB : MF) 31269fad079SSanjoy Das analyzeBlockForNullChecks(MBB, NullCheckList); 31369fad079SSanjoy Das 31469fad079SSanjoy Das if (!NullCheckList.empty()) 31569fad079SSanjoy Das rewriteNullChecks(NullCheckList); 31669fad079SSanjoy Das 31769fad079SSanjoy Das return !NullCheckList.empty(); 31869fad079SSanjoy Das } 31969fad079SSanjoy Das 320e57bf680SSanjoy Das // Return true if any register aliasing \p Reg is live-in into \p MBB. 321e57bf680SSanjoy Das static bool AnyAliasLiveIn(const TargetRegisterInfo *TRI, 322e57bf680SSanjoy Das MachineBasicBlock *MBB, unsigned Reg) { 323e57bf680SSanjoy Das for (MCRegAliasIterator AR(Reg, TRI, /*IncludeSelf*/ true); AR.isValid(); 324e57bf680SSanjoy Das ++AR) 325e57bf680SSanjoy Das if (MBB->isLiveIn(*AR)) 326e57bf680SSanjoy Das return true; 327e57bf680SSanjoy Das return false; 328e57bf680SSanjoy Das } 329e57bf680SSanjoy Das 330eef785c1SSanjoy Das ImplicitNullChecks::AliasResult 331238c9d63SBjorn Pettersson ImplicitNullChecks::areMemoryOpsAliased(const MachineInstr &MI, 332238c9d63SBjorn Pettersson const MachineInstr *PrevMI) const { 333eef785c1SSanjoy Das // If it is not memory access, skip the check. 334eef785c1SSanjoy Das if (!(PrevMI->mayStore() || PrevMI->mayLoad())) 335eef785c1SSanjoy Das return AR_NoAlias; 336eef785c1SSanjoy Das // Load-Load may alias 337eef785c1SSanjoy Das if (!(MI.mayStore() || PrevMI->mayStore())) 338eef785c1SSanjoy Das return AR_NoAlias; 339eef785c1SSanjoy Das // We lost info, conservatively alias. If it was store then no sense to 340eef785c1SSanjoy Das // continue because we won't be able to check against it further. 341eef785c1SSanjoy Das if (MI.memoperands_empty()) 342eef785c1SSanjoy Das return MI.mayStore() ? AR_WillAliasEverything : AR_MayAlias; 343eef785c1SSanjoy Das if (PrevMI->memoperands_empty()) 344eef785c1SSanjoy Das return PrevMI->mayStore() ? AR_WillAliasEverything : AR_MayAlias; 345eef785c1SSanjoy Das 346eef785c1SSanjoy Das for (MachineMemOperand *MMO1 : MI.memoperands()) { 347eef785c1SSanjoy Das // MMO1 should have a value due it comes from operation we'd like to use 348eef785c1SSanjoy Das // as implicit null check. 349eef785c1SSanjoy Das assert(MMO1->getValue() && "MMO1 should have a Value!"); 350eef785c1SSanjoy Das for (MachineMemOperand *MMO2 : PrevMI->memoperands()) { 351eef785c1SSanjoy Das if (const PseudoSourceValue *PSV = MMO2->getPseudoValue()) { 352eef785c1SSanjoy Das if (PSV->mayAlias(MFI)) 353eef785c1SSanjoy Das return AR_MayAlias; 354eef785c1SSanjoy Das continue; 355eef785c1SSanjoy Das } 356*4df8efceSNikita Popov llvm::AliasResult AAResult = AA->alias( 357*4df8efceSNikita Popov MemoryLocation::getAfter(MMO1->getValue(), MMO1->getAAInfo()), 358*4df8efceSNikita Popov MemoryLocation::getAfter(MMO2->getValue(), MMO2->getAAInfo())); 359eef785c1SSanjoy Das if (AAResult != NoAlias) 360eef785c1SSanjoy Das return AR_MayAlias; 361eef785c1SSanjoy Das } 362eef785c1SSanjoy Das } 363eef785c1SSanjoy Das return AR_NoAlias; 364eef785c1SSanjoy Das } 365eef785c1SSanjoy Das 36615e50b51SSanjoy Das ImplicitNullChecks::SuitabilityResult 367238c9d63SBjorn Pettersson ImplicitNullChecks::isSuitableMemoryOp(const MachineInstr &MI, 368238c9d63SBjorn Pettersson unsigned PointerReg, 369eef785c1SSanjoy Das ArrayRef<MachineInstr *> PrevInsts) { 370b04c181eSPhilip Reames // Implementation restriction for faulting_op insertion 371b04c181eSPhilip Reames // TODO: This could be relaxed if we find a test case which warrants it. 372b04c181eSPhilip Reames if (MI.getDesc().getNumDefs() > 1) 373b04c181eSPhilip Reames return SR_Unsuitable; 3748fbc9258SSander de Smalen 37535cb45c5SAnna Thomas if (!MI.mayLoadOrStore() || MI.isPredicable()) 376eef785c1SSanjoy Das return SR_Unsuitable; 37735cb45c5SAnna Thomas auto AM = TII->getAddrModeFromMemoryOp(MI, TRI); 37835cb45c5SAnna Thomas if (!AM) 37935cb45c5SAnna Thomas return SR_Unsuitable; 38035cb45c5SAnna Thomas auto AddrMode = *AM; 38135cb45c5SAnna Thomas const Register BaseReg = AddrMode.BaseReg, ScaledReg = AddrMode.ScaledReg; 38235cb45c5SAnna Thomas int64_t Displacement = AddrMode.Displacement; 38350fef432SSanjoy Das 3846f7737c4SAnna Thomas // We need the base of the memory instruction to be same as the register 3856f7737c4SAnna Thomas // where the null check is performed (i.e. PointerReg). 38635cb45c5SAnna Thomas if (BaseReg != PointerReg && ScaledReg != PointerReg) 38735cb45c5SAnna Thomas return SR_Unsuitable; 38835cb45c5SAnna Thomas const MachineRegisterInfo &MRI = MI.getMF()->getRegInfo(); 38935cb45c5SAnna Thomas unsigned PointerRegSizeInBits = TRI->getRegSizeInBits(PointerReg, MRI); 39035cb45c5SAnna Thomas // Bail out of the sizes of BaseReg, ScaledReg and PointerReg are not the 39135cb45c5SAnna Thomas // same. 39235cb45c5SAnna Thomas if ((BaseReg && 39335cb45c5SAnna Thomas TRI->getRegSizeInBits(BaseReg, MRI) != PointerRegSizeInBits) || 39435cb45c5SAnna Thomas (ScaledReg && 39535cb45c5SAnna Thomas TRI->getRegSizeInBits(ScaledReg, MRI) != PointerRegSizeInBits)) 3966f7737c4SAnna Thomas return SR_Unsuitable; 3976f7737c4SAnna Thomas 39835cb45c5SAnna Thomas // Returns true if RegUsedInAddr is used for calculating the displacement 39935cb45c5SAnna Thomas // depending on addressing mode. Also calculates the Displacement. 40035cb45c5SAnna Thomas auto CalculateDisplacementFromAddrMode = [&](Register RegUsedInAddr, 40135cb45c5SAnna Thomas int64_t Multiplier) { 40235cb45c5SAnna Thomas // The register can be NoRegister, which is defined as zero for all targets. 40335cb45c5SAnna Thomas // Consider instruction of interest as `movq 8(,%rdi,8), %rax`. Here the 40435cb45c5SAnna Thomas // ScaledReg is %rdi, while there is no BaseReg. 40535cb45c5SAnna Thomas if (!RegUsedInAddr) 40635cb45c5SAnna Thomas return false; 40735cb45c5SAnna Thomas assert(Multiplier && "expected to be non-zero!"); 40835cb45c5SAnna Thomas MachineInstr *ModifyingMI = nullptr; 40935cb45c5SAnna Thomas for (auto It = std::next(MachineBasicBlock::const_reverse_iterator(&MI)); 41035cb45c5SAnna Thomas It != MI.getParent()->rend(); It++) { 41135cb45c5SAnna Thomas const MachineInstr *CurrMI = &*It; 41235cb45c5SAnna Thomas if (CurrMI->modifiesRegister(RegUsedInAddr, TRI)) { 41335cb45c5SAnna Thomas ModifyingMI = const_cast<MachineInstr *>(CurrMI); 41435cb45c5SAnna Thomas break; 41535cb45c5SAnna Thomas } 41635cb45c5SAnna Thomas } 41735cb45c5SAnna Thomas if (!ModifyingMI) 41835cb45c5SAnna Thomas return false; 41935cb45c5SAnna Thomas // Check for the const value defined in register by ModifyingMI. This means 42035cb45c5SAnna Thomas // all other previous values for that register has been invalidated. 42135cb45c5SAnna Thomas int64_t ImmVal; 42235cb45c5SAnna Thomas if (!TII->getConstValDefinedInReg(*ModifyingMI, RegUsedInAddr, ImmVal)) 42335cb45c5SAnna Thomas return false; 42435cb45c5SAnna Thomas // Calculate the reg size in bits, since this is needed for bailing out in 42535cb45c5SAnna Thomas // case of overflow. 42635cb45c5SAnna Thomas int32_t RegSizeInBits = TRI->getRegSizeInBits(RegUsedInAddr, MRI); 42735cb45c5SAnna Thomas APInt ImmValC(RegSizeInBits, ImmVal, true /*IsSigned*/); 42835cb45c5SAnna Thomas APInt MultiplierC(RegSizeInBits, Multiplier); 42935cb45c5SAnna Thomas assert(MultiplierC.isStrictlyPositive() && 43035cb45c5SAnna Thomas "expected to be a positive value!"); 43135cb45c5SAnna Thomas bool IsOverflow; 43235cb45c5SAnna Thomas // Sign of the product depends on the sign of the ImmVal, since Multiplier 43335cb45c5SAnna Thomas // is always positive. 43435cb45c5SAnna Thomas APInt Product = ImmValC.smul_ov(MultiplierC, IsOverflow); 43535cb45c5SAnna Thomas if (IsOverflow) 43635cb45c5SAnna Thomas return false; 43735cb45c5SAnna Thomas APInt DisplacementC(64, Displacement, true /*isSigned*/); 43835cb45c5SAnna Thomas DisplacementC = Product.sadd_ov(DisplacementC, IsOverflow); 43935cb45c5SAnna Thomas if (IsOverflow) 44035cb45c5SAnna Thomas return false; 4418fbc9258SSander de Smalen 44235cb45c5SAnna Thomas // We only handle diplacements upto 64 bits wide. 44335cb45c5SAnna Thomas if (DisplacementC.getActiveBits() > 64) 44435cb45c5SAnna Thomas return false; 44535cb45c5SAnna Thomas Displacement = DisplacementC.getSExtValue(); 44635cb45c5SAnna Thomas return true; 44735cb45c5SAnna Thomas }; 44835cb45c5SAnna Thomas 44935cb45c5SAnna Thomas // If a register used in the address is constant, fold it's effect into the 45035cb45c5SAnna Thomas // displacement for ease of analysis. 45135cb45c5SAnna Thomas bool BaseRegIsConstVal = false, ScaledRegIsConstVal = false; 45235cb45c5SAnna Thomas if (CalculateDisplacementFromAddrMode(BaseReg, 1)) 45335cb45c5SAnna Thomas BaseRegIsConstVal = true; 45435cb45c5SAnna Thomas if (CalculateDisplacementFromAddrMode(ScaledReg, AddrMode.Scale)) 45535cb45c5SAnna Thomas ScaledRegIsConstVal = true; 45635cb45c5SAnna Thomas 45735cb45c5SAnna Thomas // The register which is not null checked should be part of the Displacement 45835cb45c5SAnna Thomas // calculation, otherwise we do not know whether the Displacement is made up 45935cb45c5SAnna Thomas // by some symbolic values. 46035cb45c5SAnna Thomas // This matters because we do not want to incorrectly assume that load from 46135cb45c5SAnna Thomas // falls in the zeroth faulting page in the "sane offset check" below. 46235cb45c5SAnna Thomas if ((BaseReg && BaseReg != PointerReg && !BaseRegIsConstVal) || 46335cb45c5SAnna Thomas (ScaledReg && ScaledReg != PointerReg && !ScaledRegIsConstVal)) 4646f7737c4SAnna Thomas return SR_Unsuitable; 4656f7737c4SAnna Thomas 4662f63cbccSSanjoy Das // We want the mem access to be issued at a sane offset from PointerReg, 4672f63cbccSSanjoy Das // so that if PointerReg is null then the access reliably page faults. 46835cb45c5SAnna Thomas if (!(-PageSize < Displacement && Displacement < PageSize)) 469eef785c1SSanjoy Das return SR_Unsuitable; 47050fef432SSanjoy Das 4710b0dc57dSSerguei Katkov // Finally, check whether the current memory access aliases with previous one. 4720b0dc57dSSerguei Katkov for (auto *PrevMI : PrevInsts) { 473eef785c1SSanjoy Das AliasResult AR = areMemoryOpsAliased(MI, PrevMI); 474eef785c1SSanjoy Das if (AR == AR_WillAliasEverything) 475eef785c1SSanjoy Das return SR_Impossible; 476eef785c1SSanjoy Das if (AR == AR_MayAlias) 4770b0dc57dSSerguei Katkov return SR_Unsuitable; 478eef785c1SSanjoy Das } 4790b0dc57dSSerguei Katkov return SR_Suitable; 48050fef432SSanjoy Das } 48150fef432SSanjoy Das 482425573a2SAnna Thomas bool ImplicitNullChecks::canDependenceHoistingClobberLiveIns( 483b1b98063SAnna Thomas MachineInstr *DependenceMI, MachineBasicBlock *NullSucc) { 4846a0ed57aSSimon Pilgrim for (const auto &DependenceMO : DependenceMI->operands()) { 485425573a2SAnna Thomas if (!(DependenceMO.isReg() && DependenceMO.getReg())) 486425573a2SAnna Thomas continue; 487425573a2SAnna Thomas 488425573a2SAnna Thomas // Make sure that we won't clobber any live ins to the sibling block by 489425573a2SAnna Thomas // hoisting Dependency. For instance, we can't hoist INST to before the 490425573a2SAnna Thomas // null check (even if it safe, and does not violate any dependencies in 491425573a2SAnna Thomas // the non_null_block) if %rdx is live in to _null_block. 492425573a2SAnna Thomas // 493425573a2SAnna Thomas // test %rcx, %rcx 494425573a2SAnna Thomas // je _null_block 495425573a2SAnna Thomas // _non_null_block: 496425573a2SAnna Thomas // %rdx = INST 497425573a2SAnna Thomas // ... 498425573a2SAnna Thomas // 499425573a2SAnna Thomas // This restriction does not apply to the faulting load inst because in 500425573a2SAnna Thomas // case the pointer loaded from is in the null page, the load will not 501425573a2SAnna Thomas // semantically execute, and affect machine state. That is, if the load 502425573a2SAnna Thomas // was loading into %rax and it faults, the value of %rax should stay the 503425573a2SAnna Thomas // same as it would have been had the load not have executed and we'd have 504425573a2SAnna Thomas // branched to NullSucc directly. 505425573a2SAnna Thomas if (AnyAliasLiveIn(TRI, NullSucc, DependenceMO.getReg())) 506425573a2SAnna Thomas return true; 507425573a2SAnna Thomas 508425573a2SAnna Thomas } 509425573a2SAnna Thomas 510425573a2SAnna Thomas // The dependence does not clobber live-ins in NullSucc block. 511425573a2SAnna Thomas return false; 512425573a2SAnna Thomas } 513425573a2SAnna Thomas 5142f63cbccSSanjoy Das bool ImplicitNullChecks::canHoistInst(MachineInstr *FaultingMI, 5152f63cbccSSanjoy Das ArrayRef<MachineInstr *> InstsSeenSoFar, 5162f63cbccSSanjoy Das MachineBasicBlock *NullSucc, 51750fef432SSanjoy Das MachineInstr *&Dependence) { 51850fef432SSanjoy Das auto DepResult = computeDependence(FaultingMI, InstsSeenSoFar); 51950fef432SSanjoy Das if (!DepResult.CanReorder) 52050fef432SSanjoy Das return false; 52150fef432SSanjoy Das 52250fef432SSanjoy Das if (!DepResult.PotentialDependence) { 52350fef432SSanjoy Das Dependence = nullptr; 52450fef432SSanjoy Das return true; 52550fef432SSanjoy Das } 52650fef432SSanjoy Das 52750fef432SSanjoy Das auto DependenceItr = *DepResult.PotentialDependence; 52850fef432SSanjoy Das auto *DependenceMI = *DependenceItr; 52950fef432SSanjoy Das 53050fef432SSanjoy Das // We don't want to reason about speculating loads. Note -- at this point 53150fef432SSanjoy Das // we should have already filtered out all of the other non-speculatable 53250fef432SSanjoy Das // things, like calls and stores. 5336ea2e81cSSerguei Katkov // We also do not want to hoist stores because it might change the memory 5346ea2e81cSSerguei Katkov // while the FaultingMI may result in faulting. 53550fef432SSanjoy Das assert(canHandle(DependenceMI) && "Should never have reached here!"); 5366ea2e81cSSerguei Katkov if (DependenceMI->mayLoadOrStore()) 53750fef432SSanjoy Das return false; 53850fef432SSanjoy Das 539b1b98063SAnna Thomas if (canDependenceHoistingClobberLiveIns(DependenceMI, NullSucc)) 54050fef432SSanjoy Das return false; 54150fef432SSanjoy Das 54250fef432SSanjoy Das auto DepDepResult = 54350fef432SSanjoy Das computeDependence(DependenceMI, {InstsSeenSoFar.begin(), DependenceItr}); 54450fef432SSanjoy Das 54550fef432SSanjoy Das if (!DepDepResult.CanReorder || DepDepResult.PotentialDependence) 54650fef432SSanjoy Das return false; 54750fef432SSanjoy Das 54850fef432SSanjoy Das Dependence = DependenceMI; 54950fef432SSanjoy Das return true; 55050fef432SSanjoy Das } 55150fef432SSanjoy Das 55269fad079SSanjoy Das /// Analyze MBB to check if its terminating branch can be turned into an 55369fad079SSanjoy Das /// implicit null check. If yes, append a description of the said null check to 55469fad079SSanjoy Das /// NullCheckList and return true, else return false. 55569fad079SSanjoy Das bool ImplicitNullChecks::analyzeBlockForNullChecks( 55669fad079SSanjoy Das MachineBasicBlock &MBB, SmallVectorImpl<NullCheck> &NullCheckList) { 557900b6335SEugene Zelenko using MachineBranchPredicate = TargetInstrInfo::MachineBranchPredicate; 55869fad079SSanjoy Das 559e8b81649SSanjoy Das MDNode *BranchMD = nullptr; 560e8b81649SSanjoy Das if (auto *BB = MBB.getBasicBlock()) 561e8b81649SSanjoy Das BranchMD = BB->getTerminator()->getMetadata(LLVMContext::MD_make_implicit); 562e8b81649SSanjoy Das 5639c41a93eSSanjoy Das if (!BranchMD) 5649c41a93eSSanjoy Das return false; 5659c41a93eSSanjoy Das 56669fad079SSanjoy Das MachineBranchPredicate MBP; 56769fad079SSanjoy Das 56871c30a14SJacques Pienaar if (TII->analyzeBranchPredicate(MBB, MBP, true)) 56969fad079SSanjoy Das return false; 57069fad079SSanjoy Das 57169fad079SSanjoy Das // Is the predicate comparing an integer to zero? 57269fad079SSanjoy Das if (!(MBP.LHS.isReg() && MBP.RHS.isImm() && MBP.RHS.getImm() == 0 && 57369fad079SSanjoy Das (MBP.Predicate == MachineBranchPredicate::PRED_NE || 57469fad079SSanjoy Das MBP.Predicate == MachineBranchPredicate::PRED_EQ))) 57569fad079SSanjoy Das return false; 57669fad079SSanjoy Das 577b04c181eSPhilip Reames // If there is a separate condition generation instruction, we chose not to 578b04c181eSPhilip Reames // transform unless we can remove both condition and consuming branch. 579b04c181eSPhilip Reames if (MBP.ConditionDef && !MBP.SingleUseCondition) 58069fad079SSanjoy Das return false; 58169fad079SSanjoy Das 58269fad079SSanjoy Das MachineBasicBlock *NotNullSucc, *NullSucc; 58369fad079SSanjoy Das 58469fad079SSanjoy Das if (MBP.Predicate == MachineBranchPredicate::PRED_NE) { 58569fad079SSanjoy Das NotNullSucc = MBP.TrueDest; 58669fad079SSanjoy Das NullSucc = MBP.FalseDest; 58769fad079SSanjoy Das } else { 58869fad079SSanjoy Das NotNullSucc = MBP.FalseDest; 58969fad079SSanjoy Das NullSucc = MBP.TrueDest; 59069fad079SSanjoy Das } 59169fad079SSanjoy Das 59269fad079SSanjoy Das // We handle the simplest case for now. We can potentially do better by using 59369fad079SSanjoy Das // the machine dominator tree. 59469fad079SSanjoy Das if (NotNullSucc->pred_size() != 1) 59569fad079SSanjoy Das return false; 59669fad079SSanjoy Das 597b04c181eSPhilip Reames const Register PointerReg = MBP.LHS.getReg(); 598b04c181eSPhilip Reames 599b04c181eSPhilip Reames if (MBP.ConditionDef) { 600e8e01143SMax Kazantsev // To prevent the invalid transformation of the following code: 601e8e01143SMax Kazantsev // 602e8e01143SMax Kazantsev // mov %rax, %rcx 603e8e01143SMax Kazantsev // test %rax, %rax 604e8e01143SMax Kazantsev // %rax = ... 605e8e01143SMax Kazantsev // je throw_npe 606e8e01143SMax Kazantsev // mov(%rcx), %r9 607e8e01143SMax Kazantsev // mov(%rax), %r10 608e8e01143SMax Kazantsev // 609e8e01143SMax Kazantsev // into: 610e8e01143SMax Kazantsev // 611e8e01143SMax Kazantsev // mov %rax, %rcx 612e8e01143SMax Kazantsev // %rax = .... 613e8e01143SMax Kazantsev // faulting_load_op("movl (%rax), %r10", throw_npe) 614e8e01143SMax Kazantsev // mov(%rcx), %r9 615e8e01143SMax Kazantsev // 616e8e01143SMax Kazantsev // we must ensure that there are no instructions between the 'test' and 617e8e01143SMax Kazantsev // conditional jump that modify %rax. 618b04c181eSPhilip Reames assert(MBP.ConditionDef->getParent() == &MBB && 619b04c181eSPhilip Reames "Should be in basic block"); 620e8e01143SMax Kazantsev 621e8e01143SMax Kazantsev for (auto I = MBB.rbegin(); MBP.ConditionDef != &*I; ++I) 622e8e01143SMax Kazantsev if (I->modifiesRegister(PointerReg, TRI)) 623e8e01143SMax Kazantsev return false; 624b04c181eSPhilip Reames } 62569fad079SSanjoy Das // Starting with a code fragment like: 62669fad079SSanjoy Das // 6279d7bb0cbSFrancis Visoiu Mistrih // test %rax, %rax 62869fad079SSanjoy Das // jne LblNotNull 62969fad079SSanjoy Das // 63069fad079SSanjoy Das // LblNull: 63169fad079SSanjoy Das // callq throw_NullPointerException 63269fad079SSanjoy Das // 63369fad079SSanjoy Das // LblNotNull: 634b7718454SSanjoy Das // Inst0 635b7718454SSanjoy Das // Inst1 636b7718454SSanjoy Das // ... 6379d7bb0cbSFrancis Visoiu Mistrih // Def = Load (%rax + <offset>) 63869fad079SSanjoy Das // ... 63969fad079SSanjoy Das // 64069fad079SSanjoy Das // 64169fad079SSanjoy Das // we want to end up with 64269fad079SSanjoy Das // 6439d7bb0cbSFrancis Visoiu Mistrih // Def = FaultingLoad (%rax + <offset>), LblNull 64469fad079SSanjoy Das // jmp LblNotNull ;; explicit or fallthrough 64569fad079SSanjoy Das // 64669fad079SSanjoy Das // LblNotNull: 647b7718454SSanjoy Das // Inst0 648b7718454SSanjoy Das // Inst1 64969fad079SSanjoy Das // ... 65069fad079SSanjoy Das // 65169fad079SSanjoy Das // LblNull: 65269fad079SSanjoy Das // callq throw_NullPointerException 65369fad079SSanjoy Das // 654ac9c5b19SSanjoy Das // 655ac9c5b19SSanjoy Das // To see why this is legal, consider the two possibilities: 656ac9c5b19SSanjoy Das // 6579d7bb0cbSFrancis Visoiu Mistrih // 1. %rax is null: since we constrain <offset> to be less than PageSize, the 658ac9c5b19SSanjoy Das // load instruction dereferences the null page, causing a segmentation 659ac9c5b19SSanjoy Das // fault. 660ac9c5b19SSanjoy Das // 6619d7bb0cbSFrancis Visoiu Mistrih // 2. %rax is not null: in this case we know that the load cannot fault, as 662ac9c5b19SSanjoy Das // otherwise the load would've faulted in the original program too and the 663ac9c5b19SSanjoy Das // original program would've been undefined. 664ac9c5b19SSanjoy Das // 665ac9c5b19SSanjoy Das // This reasoning cannot be extended to justify hoisting through arbitrary 666ac9c5b19SSanjoy Das // control flow. For instance, in the example below (in pseudo-C) 667ac9c5b19SSanjoy Das // 668ac9c5b19SSanjoy Das // if (ptr == null) { throw_npe(); unreachable; } 669ac9c5b19SSanjoy Das // if (some_cond) { return 42; } 670ac9c5b19SSanjoy Das // v = ptr->field; // LD 671ac9c5b19SSanjoy Das // ... 672ac9c5b19SSanjoy Das // 673ac9c5b19SSanjoy Das // we cannot (without code duplication) use the load marked "LD" to null check 674ac9c5b19SSanjoy Das // ptr -- clause (2) above does not apply in this case. In the above program 675ac9c5b19SSanjoy Das // the safety of ptr->field can be dependent on some_cond; and, for instance, 676ac9c5b19SSanjoy Das // ptr could be some non-null invalid reference that never gets loaded from 677ac9c5b19SSanjoy Das // because some_cond is always true. 67869fad079SSanjoy Das 6799a129807SSanjoy Das SmallVector<MachineInstr *, 8> InstsSeenSoFar; 680b7718454SSanjoy Das 6819a129807SSanjoy Das for (auto &MI : *NotNullSucc) { 6829a129807SSanjoy Das if (!canHandle(&MI) || InstsSeenSoFar.size() >= MaxInstsToConsider) 6839a129807SSanjoy Das return false; 684e57bf680SSanjoy Das 6859a129807SSanjoy Das MachineInstr *Dependence; 686eef785c1SSanjoy Das SuitabilityResult SR = isSuitableMemoryOp(MI, PointerReg, InstsSeenSoFar); 68715e50b51SSanjoy Das if (SR == SR_Impossible) 68815e50b51SSanjoy Das return false; 6892f63cbccSSanjoy Das if (SR == SR_Suitable && 690b1b98063SAnna Thomas canHoistInst(&MI, InstsSeenSoFar, NullSucc, Dependence)) { 6919cfc75c2SDuncan P. N. Exon Smith NullCheckList.emplace_back(&MI, MBP.ConditionDef, &MBB, NotNullSucc, 6929a129807SSanjoy Das NullSucc, Dependence); 693e57bf680SSanjoy Das return true; 694e57bf680SSanjoy Das } 69569fad079SSanjoy Das 69646329f60SAnna Thomas // If MI re-defines the PointerReg in a way that changes the value of 69746329f60SAnna Thomas // PointerReg if it was null, then we cannot move further. 69846329f60SAnna Thomas if (!TII->preservesZeroValueInReg(&MI, PointerReg, TRI)) 6990b0dc57dSSerguei Katkov return false; 7009a129807SSanjoy Das InstsSeenSoFar.push_back(&MI); 701b7718454SSanjoy Das } 702b7718454SSanjoy Das 70369fad079SSanjoy Das return false; 70469fad079SSanjoy Das } 70569fad079SSanjoy Das 7062f63cbccSSanjoy Das /// Wrap a machine instruction, MI, into a FAULTING machine instruction. 7072f63cbccSSanjoy Das /// The FAULTING instruction does the same load/store as MI 7082f63cbccSSanjoy Das /// (defining the same register), and branches to HandlerMBB if the mem access 7092f63cbccSSanjoy Das /// faults. The FAULTING instruction is inserted at the end of MBB. 7102f63cbccSSanjoy Das MachineInstr *ImplicitNullChecks::insertFaultingInstr( 7112f63cbccSSanjoy Das MachineInstr *MI, MachineBasicBlock *MBB, MachineBasicBlock *HandlerMBB) { 71293d608c3SSanjoy Das const unsigned NoRegister = 0; // Guaranteed to be the NoRegister value for 71393d608c3SSanjoy Das // all targets. 71493d608c3SSanjoy Das 71569fad079SSanjoy Das DebugLoc DL; 7162f63cbccSSanjoy Das unsigned NumDefs = MI->getDesc().getNumDefs(); 71793d608c3SSanjoy Das assert(NumDefs <= 1 && "other cases unhandled!"); 71869fad079SSanjoy Das 71993d608c3SSanjoy Das unsigned DefReg = NoRegister; 72093d608c3SSanjoy Das if (NumDefs != 0) { 721342273a1SCraig Topper DefReg = MI->getOperand(0).getReg(); 7225a0872c2SVedant Kumar assert(NumDefs == 1 && "expected exactly one def!"); 72393d608c3SSanjoy Das } 72469fad079SSanjoy Das 7252f63cbccSSanjoy Das FaultMaps::FaultKind FK; 7262f63cbccSSanjoy Das if (MI->mayLoad()) 7272f63cbccSSanjoy Das FK = 7282f63cbccSSanjoy Das MI->mayStore() ? FaultMaps::FaultingLoadStore : FaultMaps::FaultingLoad; 7292f63cbccSSanjoy Das else 7302f63cbccSSanjoy Das FK = FaultMaps::FaultingStore; 73169fad079SSanjoy Das 7322f63cbccSSanjoy Das auto MIB = BuildMI(MBB, DL, TII->get(TargetOpcode::FAULTING_OP), DefReg) 7332f63cbccSSanjoy Das .addImm(FK) 7342f63cbccSSanjoy Das .addMBB(HandlerMBB) 7352f63cbccSSanjoy Das .addImm(MI->getOpcode()); 7362f63cbccSSanjoy Das 737605f7795SMatthias Braun for (auto &MO : MI->uses()) { 738605f7795SMatthias Braun if (MO.isReg()) { 739605f7795SMatthias Braun MachineOperand NewMO = MO; 740605f7795SMatthias Braun if (MO.isUse()) { 741605f7795SMatthias Braun NewMO.setIsKill(false); 742605f7795SMatthias Braun } else { 743605f7795SMatthias Braun assert(MO.isDef() && "Expected def or use"); 744605f7795SMatthias Braun NewMO.setIsDead(false); 745605f7795SMatthias Braun } 746605f7795SMatthias Braun MIB.add(NewMO); 747605f7795SMatthias Braun } else { 748116bbab4SDiana Picus MIB.add(MO); 749605f7795SMatthias Braun } 750605f7795SMatthias Braun } 75169fad079SSanjoy Das 752c73c0307SChandler Carruth MIB.setMemRefs(MI->memoperands()); 75369fad079SSanjoy Das 75469fad079SSanjoy Das return MIB; 75569fad079SSanjoy Das } 75669fad079SSanjoy Das 75769fad079SSanjoy Das /// Rewrite the null checks in NullCheckList into implicit null checks. 75869fad079SSanjoy Das void ImplicitNullChecks::rewriteNullChecks( 75969fad079SSanjoy Das ArrayRef<ImplicitNullChecks::NullCheck> NullCheckList) { 76069fad079SSanjoy Das DebugLoc DL; 76169fad079SSanjoy Das 76269fad079SSanjoy Das for (auto &NC : NullCheckList) { 76369fad079SSanjoy Das // Remove the conditional branch dependent on the null check. 7641b9fc8edSMatt Arsenault unsigned BranchesRemoved = TII->removeBranch(*NC.getCheckBlock()); 76569fad079SSanjoy Das (void)BranchesRemoved; 76669fad079SSanjoy Das assert(BranchesRemoved > 0 && "expected at least one branch!"); 76769fad079SSanjoy Das 768e57bf680SSanjoy Das if (auto *DepMI = NC.getOnlyDependency()) { 769e57bf680SSanjoy Das DepMI->removeFromParent(); 770e57bf680SSanjoy Das NC.getCheckBlock()->insert(NC.getCheckBlock()->end(), DepMI); 771e57bf680SSanjoy Das } 772e57bf680SSanjoy Das 7732f63cbccSSanjoy Das // Insert a faulting instruction where the conditional branch was 7742f63cbccSSanjoy Das // originally. We check earlier ensures that this bit of code motion 7752f63cbccSSanjoy Das // is legal. We do not touch the successors list for any basic block 7762f63cbccSSanjoy Das // since we haven't changed control flow, we've just made it implicit. 7772f63cbccSSanjoy Das MachineInstr *FaultingInstr = insertFaultingInstr( 778e173b9aeSSanjoy Das NC.getMemOperation(), NC.getCheckBlock(), NC.getNullSucc()); 77926dab3a4SQuentin Colombet // Now the values defined by MemOperation, if any, are live-in of 78026dab3a4SQuentin Colombet // the block of MemOperation. 7812f63cbccSSanjoy Das // The original operation may define implicit-defs alongside 7822f63cbccSSanjoy Das // the value. 783e173b9aeSSanjoy Das MachineBasicBlock *MBB = NC.getMemOperation()->getParent(); 7842f63cbccSSanjoy Das for (const MachineOperand &MO : FaultingInstr->operands()) { 78526dab3a4SQuentin Colombet if (!MO.isReg() || !MO.isDef()) 78626dab3a4SQuentin Colombet continue; 7870c476111SDaniel Sanders Register Reg = MO.getReg(); 78826dab3a4SQuentin Colombet if (!Reg || MBB->isLiveIn(Reg)) 78926dab3a4SQuentin Colombet continue; 79012b69919SQuentin Colombet MBB->addLiveIn(Reg); 79112b69919SQuentin Colombet } 792e57bf680SSanjoy Das 793e57bf680SSanjoy Das if (auto *DepMI = NC.getOnlyDependency()) { 794e57bf680SSanjoy Das for (auto &MO : DepMI->operands()) { 795f8c0cfc2SJonas Paulsson if (!MO.isReg() || !MO.getReg() || !MO.isDef() || MO.isDead()) 796e57bf680SSanjoy Das continue; 797e57bf680SSanjoy Das if (!NC.getNotNullSucc()->isLiveIn(MO.getReg())) 798e57bf680SSanjoy Das NC.getNotNullSucc()->addLiveIn(MO.getReg()); 799e57bf680SSanjoy Das } 800e57bf680SSanjoy Das } 801e57bf680SSanjoy Das 802e173b9aeSSanjoy Das NC.getMemOperation()->eraseFromParent(); 803b04c181eSPhilip Reames if (auto *CheckOp = NC.getCheckOperation()) 804b04c181eSPhilip Reames CheckOp->eraseFromParent(); 80569fad079SSanjoy Das 806b04c181eSPhilip Reames // Insert an *unconditional* branch to not-null successor - we expect 807b04c181eSPhilip Reames // block placement to remove fallthroughs later. 808e8e0f5caSMatt Arsenault TII->insertBranch(*NC.getCheckBlock(), NC.getNotNullSucc(), nullptr, 809e173b9aeSSanjoy Das /*Cond=*/None, DL); 81069fad079SSanjoy Das 8118ee6a30bSSanjoy Das NumImplicitNullChecks++; 81269fad079SSanjoy Das } 81369fad079SSanjoy Das } 81469fad079SSanjoy Das 81569fad079SSanjoy Das char ImplicitNullChecks::ID = 0; 816900b6335SEugene Zelenko 81769fad079SSanjoy Das char &llvm::ImplicitNullChecksID = ImplicitNullChecks::ID; 818900b6335SEugene Zelenko 8191527baabSMatthias Braun INITIALIZE_PASS_BEGIN(ImplicitNullChecks, DEBUG_TYPE, 82069fad079SSanjoy Das "Implicit null checks", false, false) 821e57bf680SSanjoy Das INITIALIZE_PASS_DEPENDENCY(AAResultsWrapperPass) 8221527baabSMatthias Braun INITIALIZE_PASS_END(ImplicitNullChecks, DEBUG_TYPE, 82369fad079SSanjoy Das "Implicit null checks", false, false) 824