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 
DependenceResult__anonb1e90f8b0111::ImplicitNullChecks::DependenceResult1039a129807SSanjoy 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:
NullCheck(MachineInstr * memOperation,MachineInstr * checkOperation,MachineBasicBlock * checkBlock,MachineBasicBlock * notNullSucc,MachineBasicBlock * nullSucc,MachineInstr * onlyDependency)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 
getMemOperation() const151e173b9aeSSanjoy Das     MachineInstr *getMemOperation() const { return MemOperation; }
152e173b9aeSSanjoy Das 
getCheckOperation() const153e173b9aeSSanjoy Das     MachineInstr *getCheckOperation() const { return CheckOperation; }
154e173b9aeSSanjoy Das 
getCheckBlock() const155e173b9aeSSanjoy Das     MachineBasicBlock *getCheckBlock() const { return CheckBlock; }
156e173b9aeSSanjoy Das 
getNotNullSucc() const157e173b9aeSSanjoy Das     MachineBasicBlock *getNotNullSucc() const { return NotNullSucc; }
158e173b9aeSSanjoy Das 
getNullSucc() const159e173b9aeSSanjoy Das     MachineBasicBlock *getNullSucc() const { return NullSucc; }
160e57bf680SSanjoy Das 
getOnlyDependency() const161e57bf680SSanjoy 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 
ImplicitNullChecks()21969fad079SSanjoy Das   ImplicitNullChecks() : MachineFunctionPass(ID) {
22069fad079SSanjoy Das     initializeImplicitNullChecksPass(*PassRegistry::getPassRegistry());
22169fad079SSanjoy Das   }
22269fad079SSanjoy Das 
22369fad079SSanjoy Das   bool runOnMachineFunction(MachineFunction &MF) override;
224900b6335SEugene Zelenko 
getAnalysisUsage(AnalysisUsage & AU) const225e57bf680SSanjoy Das   void getAnalysisUsage(AnalysisUsage &AU) const override {
226e57bf680SSanjoy Das     AU.addRequired<AAResultsWrapperPass>();
227e57bf680SSanjoy Das     MachineFunctionPass::getAnalysisUsage(AU);
228e57bf680SSanjoy Das   }
229ad154c83SDerek Schuff 
getRequiredProperties() const230ad154c83SDerek 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 
canHandle(const MachineInstr * MI)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 
24526bd534aSKazu Hirata   assert(llvm::none_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
computeDependence(const MachineInstr * MI,ArrayRef<MachineInstr * > Block)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 
canReorder(const MachineInstr * A,const MachineInstr * B)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 
runOnMachineFunction(MachineFunction & MF)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.
AnyAliasLiveIn(const TargetRegisterInfo * TRI,MachineBasicBlock * MBB,unsigned Reg)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
areMemoryOpsAliased(const MachineInstr & MI,const MachineInstr * PrevMI) const331238c9d63SBjorn 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       }
356d0660797Sdfukalov       if (!AA->isNoAlias(
3574df8efceSNikita Popov               MemoryLocation::getAfter(MMO1->getValue(), MMO1->getAAInfo()),
358d0660797Sdfukalov               MemoryLocation::getAfter(MMO2->getValue(), MMO2->getAAInfo())))
359eef785c1SSanjoy Das         return AR_MayAlias;
360eef785c1SSanjoy Das     }
361eef785c1SSanjoy Das   }
362eef785c1SSanjoy Das   return AR_NoAlias;
363eef785c1SSanjoy Das }
364eef785c1SSanjoy Das 
36515e50b51SSanjoy Das ImplicitNullChecks::SuitabilityResult
isSuitableMemoryOp(const MachineInstr & MI,unsigned PointerReg,ArrayRef<MachineInstr * > PrevInsts)366238c9d63SBjorn Pettersson ImplicitNullChecks::isSuitableMemoryOp(const MachineInstr &MI,
367238c9d63SBjorn Pettersson                                        unsigned PointerReg,
368eef785c1SSanjoy Das                                        ArrayRef<MachineInstr *> PrevInsts) {
369b04c181eSPhilip Reames   // Implementation restriction for faulting_op insertion
370b04c181eSPhilip Reames   // TODO: This could be relaxed if we find a test case which warrants it.
371b04c181eSPhilip Reames   if (MI.getDesc().getNumDefs() > 1)
372b04c181eSPhilip Reames    return SR_Unsuitable;
3738fbc9258SSander de Smalen 
37435cb45c5SAnna Thomas   if (!MI.mayLoadOrStore() || MI.isPredicable())
375eef785c1SSanjoy Das     return SR_Unsuitable;
37635cb45c5SAnna Thomas   auto AM = TII->getAddrModeFromMemoryOp(MI, TRI);
37735cb45c5SAnna Thomas   if (!AM)
37835cb45c5SAnna Thomas     return SR_Unsuitable;
37935cb45c5SAnna Thomas   auto AddrMode = *AM;
38035cb45c5SAnna Thomas   const Register BaseReg = AddrMode.BaseReg, ScaledReg = AddrMode.ScaledReg;
38135cb45c5SAnna Thomas   int64_t Displacement = AddrMode.Displacement;
38250fef432SSanjoy Das 
3836f7737c4SAnna Thomas   // We need the base of the memory instruction to be same as the register
3846f7737c4SAnna Thomas   // where the null check is performed (i.e. PointerReg).
38535cb45c5SAnna Thomas   if (BaseReg != PointerReg && ScaledReg != PointerReg)
38635cb45c5SAnna Thomas     return SR_Unsuitable;
38735cb45c5SAnna Thomas   const MachineRegisterInfo &MRI = MI.getMF()->getRegInfo();
38835cb45c5SAnna Thomas   unsigned PointerRegSizeInBits = TRI->getRegSizeInBits(PointerReg, MRI);
38935cb45c5SAnna Thomas   // Bail out of the sizes of BaseReg, ScaledReg and PointerReg are not the
39035cb45c5SAnna Thomas   // same.
39135cb45c5SAnna Thomas   if ((BaseReg &&
39235cb45c5SAnna Thomas        TRI->getRegSizeInBits(BaseReg, MRI) != PointerRegSizeInBits) ||
39335cb45c5SAnna Thomas       (ScaledReg &&
39435cb45c5SAnna Thomas        TRI->getRegSizeInBits(ScaledReg, MRI) != PointerRegSizeInBits))
3956f7737c4SAnna Thomas     return SR_Unsuitable;
3966f7737c4SAnna Thomas 
39735cb45c5SAnna Thomas   // Returns true if RegUsedInAddr is used for calculating the displacement
39835cb45c5SAnna Thomas   // depending on addressing mode. Also calculates the Displacement.
39935cb45c5SAnna Thomas   auto CalculateDisplacementFromAddrMode = [&](Register RegUsedInAddr,
40035cb45c5SAnna Thomas                                                int64_t Multiplier) {
40135cb45c5SAnna Thomas     // The register can be NoRegister, which is defined as zero for all targets.
40235cb45c5SAnna Thomas     // Consider instruction of interest as `movq 8(,%rdi,8), %rax`. Here the
40335cb45c5SAnna Thomas     // ScaledReg is %rdi, while there is no BaseReg.
40435cb45c5SAnna Thomas     if (!RegUsedInAddr)
40535cb45c5SAnna Thomas       return false;
40635cb45c5SAnna Thomas     assert(Multiplier && "expected to be non-zero!");
40735cb45c5SAnna Thomas     MachineInstr *ModifyingMI = nullptr;
40835cb45c5SAnna Thomas     for (auto It = std::next(MachineBasicBlock::const_reverse_iterator(&MI));
40935cb45c5SAnna Thomas          It != MI.getParent()->rend(); It++) {
41035cb45c5SAnna Thomas       const MachineInstr *CurrMI = &*It;
41135cb45c5SAnna Thomas       if (CurrMI->modifiesRegister(RegUsedInAddr, TRI)) {
41235cb45c5SAnna Thomas         ModifyingMI = const_cast<MachineInstr *>(CurrMI);
41335cb45c5SAnna Thomas         break;
41435cb45c5SAnna Thomas       }
41535cb45c5SAnna Thomas     }
41635cb45c5SAnna Thomas     if (!ModifyingMI)
41735cb45c5SAnna Thomas       return false;
41835cb45c5SAnna Thomas     // Check for the const value defined in register by ModifyingMI. This means
41935cb45c5SAnna Thomas     // all other previous values for that register has been invalidated.
42035cb45c5SAnna Thomas     int64_t ImmVal;
42135cb45c5SAnna Thomas     if (!TII->getConstValDefinedInReg(*ModifyingMI, RegUsedInAddr, ImmVal))
42235cb45c5SAnna Thomas       return false;
42335cb45c5SAnna Thomas     // Calculate the reg size in bits, since this is needed for bailing out in
42435cb45c5SAnna Thomas     // case of overflow.
42535cb45c5SAnna Thomas     int32_t RegSizeInBits = TRI->getRegSizeInBits(RegUsedInAddr, MRI);
42635cb45c5SAnna Thomas     APInt ImmValC(RegSizeInBits, ImmVal, true /*IsSigned*/);
42735cb45c5SAnna Thomas     APInt MultiplierC(RegSizeInBits, Multiplier);
42835cb45c5SAnna Thomas     assert(MultiplierC.isStrictlyPositive() &&
42935cb45c5SAnna Thomas            "expected to be a positive value!");
43035cb45c5SAnna Thomas     bool IsOverflow;
43135cb45c5SAnna Thomas     // Sign of the product depends on the sign of the ImmVal, since Multiplier
43235cb45c5SAnna Thomas     // is always positive.
43335cb45c5SAnna Thomas     APInt Product = ImmValC.smul_ov(MultiplierC, IsOverflow);
43435cb45c5SAnna Thomas     if (IsOverflow)
43535cb45c5SAnna Thomas       return false;
43635cb45c5SAnna Thomas     APInt DisplacementC(64, Displacement, true /*isSigned*/);
43735cb45c5SAnna Thomas     DisplacementC = Product.sadd_ov(DisplacementC, IsOverflow);
43835cb45c5SAnna Thomas     if (IsOverflow)
43935cb45c5SAnna Thomas       return false;
4408fbc9258SSander de Smalen 
44135cb45c5SAnna Thomas     // We only handle diplacements upto 64 bits wide.
44235cb45c5SAnna Thomas     if (DisplacementC.getActiveBits() > 64)
44335cb45c5SAnna Thomas       return false;
44435cb45c5SAnna Thomas     Displacement = DisplacementC.getSExtValue();
44535cb45c5SAnna Thomas     return true;
44635cb45c5SAnna Thomas   };
44735cb45c5SAnna Thomas 
44835cb45c5SAnna Thomas   // If a register used in the address is constant, fold it's effect into the
44935cb45c5SAnna Thomas   // displacement for ease of analysis.
45035cb45c5SAnna Thomas   bool BaseRegIsConstVal = false, ScaledRegIsConstVal = false;
45135cb45c5SAnna Thomas   if (CalculateDisplacementFromAddrMode(BaseReg, 1))
45235cb45c5SAnna Thomas     BaseRegIsConstVal = true;
45335cb45c5SAnna Thomas   if (CalculateDisplacementFromAddrMode(ScaledReg, AddrMode.Scale))
45435cb45c5SAnna Thomas     ScaledRegIsConstVal = true;
45535cb45c5SAnna Thomas 
45635cb45c5SAnna Thomas   // The register which is not null checked should be part of the Displacement
45735cb45c5SAnna Thomas   // calculation, otherwise we do not know whether the Displacement is made up
45835cb45c5SAnna Thomas   // by some symbolic values.
45935cb45c5SAnna Thomas   // This matters because we do not want to incorrectly assume that load from
46035cb45c5SAnna Thomas   // falls in the zeroth faulting page in the "sane offset check" below.
46135cb45c5SAnna Thomas   if ((BaseReg && BaseReg != PointerReg && !BaseRegIsConstVal) ||
46235cb45c5SAnna Thomas       (ScaledReg && ScaledReg != PointerReg && !ScaledRegIsConstVal))
4636f7737c4SAnna Thomas     return SR_Unsuitable;
4646f7737c4SAnna Thomas 
4652f63cbccSSanjoy Das   // We want the mem access to be issued at a sane offset from PointerReg,
4662f63cbccSSanjoy Das   // so that if PointerReg is null then the access reliably page faults.
46735cb45c5SAnna Thomas   if (!(-PageSize < Displacement && Displacement < PageSize))
468eef785c1SSanjoy Das     return SR_Unsuitable;
46950fef432SSanjoy Das 
4700b0dc57dSSerguei Katkov   // Finally, check whether the current memory access aliases with previous one.
4710b0dc57dSSerguei Katkov   for (auto *PrevMI : PrevInsts) {
472eef785c1SSanjoy Das     AliasResult AR = areMemoryOpsAliased(MI, PrevMI);
473eef785c1SSanjoy Das     if (AR == AR_WillAliasEverything)
474eef785c1SSanjoy Das       return SR_Impossible;
475eef785c1SSanjoy Das     if (AR == AR_MayAlias)
4760b0dc57dSSerguei Katkov       return SR_Unsuitable;
477eef785c1SSanjoy Das   }
4780b0dc57dSSerguei Katkov   return SR_Suitable;
47950fef432SSanjoy Das }
48050fef432SSanjoy Das 
canDependenceHoistingClobberLiveIns(MachineInstr * DependenceMI,MachineBasicBlock * NullSucc)481425573a2SAnna Thomas bool ImplicitNullChecks::canDependenceHoistingClobberLiveIns(
482b1b98063SAnna Thomas     MachineInstr *DependenceMI, MachineBasicBlock *NullSucc) {
4836a0ed57aSSimon Pilgrim   for (const auto &DependenceMO : DependenceMI->operands()) {
484425573a2SAnna Thomas     if (!(DependenceMO.isReg() && DependenceMO.getReg()))
485425573a2SAnna Thomas       continue;
486425573a2SAnna Thomas 
487425573a2SAnna Thomas     // Make sure that we won't clobber any live ins to the sibling block by
488425573a2SAnna Thomas     // hoisting Dependency.  For instance, we can't hoist INST to before the
489425573a2SAnna Thomas     // null check (even if it safe, and does not violate any dependencies in
490425573a2SAnna Thomas     // the non_null_block) if %rdx is live in to _null_block.
491425573a2SAnna Thomas     //
492425573a2SAnna Thomas     //    test %rcx, %rcx
493425573a2SAnna Thomas     //    je _null_block
494425573a2SAnna Thomas     //  _non_null_block:
495425573a2SAnna Thomas     //    %rdx = INST
496425573a2SAnna Thomas     //    ...
497425573a2SAnna Thomas     //
498425573a2SAnna Thomas     // This restriction does not apply to the faulting load inst because in
499425573a2SAnna Thomas     // case the pointer loaded from is in the null page, the load will not
500425573a2SAnna Thomas     // semantically execute, and affect machine state.  That is, if the load
501425573a2SAnna Thomas     // was loading into %rax and it faults, the value of %rax should stay the
502425573a2SAnna Thomas     // same as it would have been had the load not have executed and we'd have
503425573a2SAnna Thomas     // branched to NullSucc directly.
504425573a2SAnna Thomas     if (AnyAliasLiveIn(TRI, NullSucc, DependenceMO.getReg()))
505425573a2SAnna Thomas       return true;
506425573a2SAnna Thomas 
507425573a2SAnna Thomas   }
508425573a2SAnna Thomas 
509425573a2SAnna Thomas   // The dependence does not clobber live-ins in NullSucc block.
510425573a2SAnna Thomas   return false;
511425573a2SAnna Thomas }
512425573a2SAnna Thomas 
canHoistInst(MachineInstr * FaultingMI,ArrayRef<MachineInstr * > InstsSeenSoFar,MachineBasicBlock * NullSucc,MachineInstr * & Dependence)5132f63cbccSSanjoy Das bool ImplicitNullChecks::canHoistInst(MachineInstr *FaultingMI,
5142f63cbccSSanjoy Das                                       ArrayRef<MachineInstr *> InstsSeenSoFar,
5152f63cbccSSanjoy Das                                       MachineBasicBlock *NullSucc,
51650fef432SSanjoy Das                                       MachineInstr *&Dependence) {
51750fef432SSanjoy Das   auto DepResult = computeDependence(FaultingMI, InstsSeenSoFar);
51850fef432SSanjoy Das   if (!DepResult.CanReorder)
51950fef432SSanjoy Das     return false;
52050fef432SSanjoy Das 
52150fef432SSanjoy Das   if (!DepResult.PotentialDependence) {
52250fef432SSanjoy Das     Dependence = nullptr;
52350fef432SSanjoy Das     return true;
52450fef432SSanjoy Das   }
52550fef432SSanjoy Das 
52650fef432SSanjoy Das   auto DependenceItr = *DepResult.PotentialDependence;
52750fef432SSanjoy Das   auto *DependenceMI = *DependenceItr;
52850fef432SSanjoy Das 
52950fef432SSanjoy Das   // We don't want to reason about speculating loads.  Note -- at this point
53050fef432SSanjoy Das   // we should have already filtered out all of the other non-speculatable
53150fef432SSanjoy Das   // things, like calls and stores.
5326ea2e81cSSerguei Katkov   // We also do not want to hoist stores because it might change the memory
5336ea2e81cSSerguei Katkov   // while the FaultingMI may result in faulting.
53450fef432SSanjoy Das   assert(canHandle(DependenceMI) && "Should never have reached here!");
5356ea2e81cSSerguei Katkov   if (DependenceMI->mayLoadOrStore())
53650fef432SSanjoy Das     return false;
53750fef432SSanjoy Das 
538b1b98063SAnna Thomas   if (canDependenceHoistingClobberLiveIns(DependenceMI, NullSucc))
53950fef432SSanjoy Das     return false;
54050fef432SSanjoy Das 
54150fef432SSanjoy Das   auto DepDepResult =
54250fef432SSanjoy Das       computeDependence(DependenceMI, {InstsSeenSoFar.begin(), DependenceItr});
54350fef432SSanjoy Das 
54450fef432SSanjoy Das   if (!DepDepResult.CanReorder || DepDepResult.PotentialDependence)
54550fef432SSanjoy Das     return false;
54650fef432SSanjoy Das 
54750fef432SSanjoy Das   Dependence = DependenceMI;
54850fef432SSanjoy Das   return true;
54950fef432SSanjoy Das }
55050fef432SSanjoy Das 
55169fad079SSanjoy Das /// Analyze MBB to check if its terminating branch can be turned into an
55269fad079SSanjoy Das /// implicit null check.  If yes, append a description of the said null check to
55369fad079SSanjoy Das /// NullCheckList and return true, else return false.
analyzeBlockForNullChecks(MachineBasicBlock & MBB,SmallVectorImpl<NullCheck> & NullCheckList)55469fad079SSanjoy Das bool ImplicitNullChecks::analyzeBlockForNullChecks(
55569fad079SSanjoy Das     MachineBasicBlock &MBB, SmallVectorImpl<NullCheck> &NullCheckList) {
556900b6335SEugene Zelenko   using MachineBranchPredicate = TargetInstrInfo::MachineBranchPredicate;
55769fad079SSanjoy Das 
558e8b81649SSanjoy Das   MDNode *BranchMD = nullptr;
559e8b81649SSanjoy Das   if (auto *BB = MBB.getBasicBlock())
560e8b81649SSanjoy Das     BranchMD = BB->getTerminator()->getMetadata(LLVMContext::MD_make_implicit);
561e8b81649SSanjoy Das 
5629c41a93eSSanjoy Das   if (!BranchMD)
5639c41a93eSSanjoy Das     return false;
5649c41a93eSSanjoy Das 
56569fad079SSanjoy Das   MachineBranchPredicate MBP;
56669fad079SSanjoy Das 
56771c30a14SJacques Pienaar   if (TII->analyzeBranchPredicate(MBB, MBP, true))
56869fad079SSanjoy Das     return false;
56969fad079SSanjoy Das 
57069fad079SSanjoy Das   // Is the predicate comparing an integer to zero?
57169fad079SSanjoy Das   if (!(MBP.LHS.isReg() && MBP.RHS.isImm() && MBP.RHS.getImm() == 0 &&
57269fad079SSanjoy Das         (MBP.Predicate == MachineBranchPredicate::PRED_NE ||
57369fad079SSanjoy Das          MBP.Predicate == MachineBranchPredicate::PRED_EQ)))
57469fad079SSanjoy Das     return false;
57569fad079SSanjoy Das 
576b04c181eSPhilip Reames   // If there is a separate condition generation instruction, we chose not to
577b04c181eSPhilip Reames   // transform unless we can remove both condition and consuming branch.
578b04c181eSPhilip Reames   if (MBP.ConditionDef && !MBP.SingleUseCondition)
57969fad079SSanjoy Das     return false;
58069fad079SSanjoy Das 
58169fad079SSanjoy Das   MachineBasicBlock *NotNullSucc, *NullSucc;
58269fad079SSanjoy Das 
58369fad079SSanjoy Das   if (MBP.Predicate == MachineBranchPredicate::PRED_NE) {
58469fad079SSanjoy Das     NotNullSucc = MBP.TrueDest;
58569fad079SSanjoy Das     NullSucc = MBP.FalseDest;
58669fad079SSanjoy Das   } else {
58769fad079SSanjoy Das     NotNullSucc = MBP.FalseDest;
58869fad079SSanjoy Das     NullSucc = MBP.TrueDest;
58969fad079SSanjoy Das   }
59069fad079SSanjoy Das 
59169fad079SSanjoy Das   // We handle the simplest case for now.  We can potentially do better by using
59269fad079SSanjoy Das   // the machine dominator tree.
59369fad079SSanjoy Das   if (NotNullSucc->pred_size() != 1)
59469fad079SSanjoy Das     return false;
59569fad079SSanjoy Das 
596b04c181eSPhilip Reames   const Register PointerReg = MBP.LHS.getReg();
597b04c181eSPhilip Reames 
598b04c181eSPhilip Reames   if (MBP.ConditionDef) {
599e8e01143SMax Kazantsev     // To prevent the invalid transformation of the following code:
600e8e01143SMax Kazantsev     //
601e8e01143SMax Kazantsev     //   mov %rax, %rcx
602e8e01143SMax Kazantsev     //   test %rax, %rax
603e8e01143SMax Kazantsev     //   %rax = ...
604e8e01143SMax Kazantsev     //   je throw_npe
605e8e01143SMax Kazantsev     //   mov(%rcx), %r9
606e8e01143SMax Kazantsev     //   mov(%rax), %r10
607e8e01143SMax Kazantsev     //
608e8e01143SMax Kazantsev     // into:
609e8e01143SMax Kazantsev     //
610e8e01143SMax Kazantsev     //   mov %rax, %rcx
611e8e01143SMax Kazantsev     //   %rax = ....
612e8e01143SMax Kazantsev     //   faulting_load_op("movl (%rax), %r10", throw_npe)
613e8e01143SMax Kazantsev     //   mov(%rcx), %r9
614e8e01143SMax Kazantsev     //
615e8e01143SMax Kazantsev     // we must ensure that there are no instructions between the 'test' and
616e8e01143SMax Kazantsev     // conditional jump that modify %rax.
617b04c181eSPhilip Reames     assert(MBP.ConditionDef->getParent() ==  &MBB &&
618b04c181eSPhilip Reames            "Should be in basic block");
619e8e01143SMax Kazantsev 
620e8e01143SMax Kazantsev     for (auto I = MBB.rbegin(); MBP.ConditionDef != &*I; ++I)
621e8e01143SMax Kazantsev       if (I->modifiesRegister(PointerReg, TRI))
622e8e01143SMax Kazantsev         return false;
623b04c181eSPhilip Reames   }
62469fad079SSanjoy Das   // Starting with a code fragment like:
62569fad079SSanjoy Das   //
6269d7bb0cbSFrancis Visoiu Mistrih   //   test %rax, %rax
62769fad079SSanjoy Das   //   jne LblNotNull
62869fad079SSanjoy Das   //
62969fad079SSanjoy Das   //  LblNull:
63069fad079SSanjoy Das   //   callq throw_NullPointerException
63169fad079SSanjoy Das   //
63269fad079SSanjoy Das   //  LblNotNull:
633b7718454SSanjoy Das   //   Inst0
634b7718454SSanjoy Das   //   Inst1
635b7718454SSanjoy Das   //   ...
6369d7bb0cbSFrancis Visoiu Mistrih   //   Def = Load (%rax + <offset>)
63769fad079SSanjoy Das   //   ...
63869fad079SSanjoy Das   //
63969fad079SSanjoy Das   //
64069fad079SSanjoy Das   // we want to end up with
64169fad079SSanjoy Das   //
6429d7bb0cbSFrancis Visoiu Mistrih   //   Def = FaultingLoad (%rax + <offset>), LblNull
64369fad079SSanjoy Das   //   jmp LblNotNull ;; explicit or fallthrough
64469fad079SSanjoy Das   //
64569fad079SSanjoy Das   //  LblNotNull:
646b7718454SSanjoy Das   //   Inst0
647b7718454SSanjoy Das   //   Inst1
64869fad079SSanjoy Das   //   ...
64969fad079SSanjoy Das   //
65069fad079SSanjoy Das   //  LblNull:
65169fad079SSanjoy Das   //   callq throw_NullPointerException
65269fad079SSanjoy Das   //
653ac9c5b19SSanjoy Das   //
654ac9c5b19SSanjoy Das   // To see why this is legal, consider the two possibilities:
655ac9c5b19SSanjoy Das   //
6569d7bb0cbSFrancis Visoiu Mistrih   //  1. %rax is null: since we constrain <offset> to be less than PageSize, the
657ac9c5b19SSanjoy Das   //     load instruction dereferences the null page, causing a segmentation
658ac9c5b19SSanjoy Das   //     fault.
659ac9c5b19SSanjoy Das   //
6609d7bb0cbSFrancis Visoiu Mistrih   //  2. %rax is not null: in this case we know that the load cannot fault, as
661ac9c5b19SSanjoy Das   //     otherwise the load would've faulted in the original program too and the
662ac9c5b19SSanjoy Das   //     original program would've been undefined.
663ac9c5b19SSanjoy Das   //
664ac9c5b19SSanjoy Das   // This reasoning cannot be extended to justify hoisting through arbitrary
665ac9c5b19SSanjoy Das   // control flow.  For instance, in the example below (in pseudo-C)
666ac9c5b19SSanjoy Das   //
667ac9c5b19SSanjoy Das   //    if (ptr == null) { throw_npe(); unreachable; }
668ac9c5b19SSanjoy Das   //    if (some_cond) { return 42; }
669ac9c5b19SSanjoy Das   //    v = ptr->field;  // LD
670ac9c5b19SSanjoy Das   //    ...
671ac9c5b19SSanjoy Das   //
672ac9c5b19SSanjoy Das   // we cannot (without code duplication) use the load marked "LD" to null check
673ac9c5b19SSanjoy Das   // ptr -- clause (2) above does not apply in this case.  In the above program
674ac9c5b19SSanjoy Das   // the safety of ptr->field can be dependent on some_cond; and, for instance,
675ac9c5b19SSanjoy Das   // ptr could be some non-null invalid reference that never gets loaded from
676ac9c5b19SSanjoy Das   // because some_cond is always true.
67769fad079SSanjoy Das 
6789a129807SSanjoy Das   SmallVector<MachineInstr *, 8> InstsSeenSoFar;
679b7718454SSanjoy Das 
6809a129807SSanjoy Das   for (auto &MI : *NotNullSucc) {
6819a129807SSanjoy Das     if (!canHandle(&MI) || InstsSeenSoFar.size() >= MaxInstsToConsider)
6829a129807SSanjoy Das       return false;
683e57bf680SSanjoy Das 
6849a129807SSanjoy Das     MachineInstr *Dependence;
685eef785c1SSanjoy Das     SuitabilityResult SR = isSuitableMemoryOp(MI, PointerReg, InstsSeenSoFar);
68615e50b51SSanjoy Das     if (SR == SR_Impossible)
68715e50b51SSanjoy Das       return false;
6882f63cbccSSanjoy Das     if (SR == SR_Suitable &&
689b1b98063SAnna Thomas         canHoistInst(&MI, InstsSeenSoFar, NullSucc, Dependence)) {
6909cfc75c2SDuncan P. N. Exon Smith       NullCheckList.emplace_back(&MI, MBP.ConditionDef, &MBB, NotNullSucc,
6919a129807SSanjoy Das                                  NullSucc, Dependence);
692e57bf680SSanjoy Das       return true;
693e57bf680SSanjoy Das     }
69469fad079SSanjoy Das 
69546329f60SAnna Thomas     // If MI re-defines the PointerReg in a way that changes the value of
69646329f60SAnna Thomas     // PointerReg if it was null, then we cannot move further.
69746329f60SAnna Thomas     if (!TII->preservesZeroValueInReg(&MI, PointerReg, TRI))
6980b0dc57dSSerguei Katkov       return false;
6999a129807SSanjoy Das     InstsSeenSoFar.push_back(&MI);
700b7718454SSanjoy Das   }
701b7718454SSanjoy Das 
70269fad079SSanjoy Das   return false;
70369fad079SSanjoy Das }
70469fad079SSanjoy Das 
7052f63cbccSSanjoy Das /// Wrap a machine instruction, MI, into a FAULTING machine instruction.
7062f63cbccSSanjoy Das /// The FAULTING instruction does the same load/store as MI
7072f63cbccSSanjoy Das /// (defining the same register), and branches to HandlerMBB if the mem access
7082f63cbccSSanjoy Das /// faults.  The FAULTING instruction is inserted at the end of MBB.
insertFaultingInstr(MachineInstr * MI,MachineBasicBlock * MBB,MachineBasicBlock * HandlerMBB)7092f63cbccSSanjoy Das MachineInstr *ImplicitNullChecks::insertFaultingInstr(
7102f63cbccSSanjoy Das     MachineInstr *MI, MachineBasicBlock *MBB, MachineBasicBlock *HandlerMBB) {
71193d608c3SSanjoy Das   const unsigned NoRegister = 0; // Guaranteed to be the NoRegister value for
71293d608c3SSanjoy Das                                  // all targets.
71393d608c3SSanjoy Das 
71469fad079SSanjoy Das   DebugLoc DL;
7152f63cbccSSanjoy Das   unsigned NumDefs = MI->getDesc().getNumDefs();
71693d608c3SSanjoy Das   assert(NumDefs <= 1 && "other cases unhandled!");
71769fad079SSanjoy Das 
71893d608c3SSanjoy Das   unsigned DefReg = NoRegister;
71993d608c3SSanjoy Das   if (NumDefs != 0) {
720342273a1SCraig Topper     DefReg = MI->getOperand(0).getReg();
7215a0872c2SVedant Kumar     assert(NumDefs == 1 && "expected exactly one def!");
72293d608c3SSanjoy Das   }
72369fad079SSanjoy Das 
7242f63cbccSSanjoy Das   FaultMaps::FaultKind FK;
7252f63cbccSSanjoy Das   if (MI->mayLoad())
7262f63cbccSSanjoy Das     FK =
7272f63cbccSSanjoy Das         MI->mayStore() ? FaultMaps::FaultingLoadStore : FaultMaps::FaultingLoad;
7282f63cbccSSanjoy Das   else
7292f63cbccSSanjoy Das     FK = FaultMaps::FaultingStore;
73069fad079SSanjoy Das 
7312f63cbccSSanjoy Das   auto MIB = BuildMI(MBB, DL, TII->get(TargetOpcode::FAULTING_OP), DefReg)
7322f63cbccSSanjoy Das                  .addImm(FK)
7332f63cbccSSanjoy Das                  .addMBB(HandlerMBB)
7342f63cbccSSanjoy Das                  .addImm(MI->getOpcode());
7352f63cbccSSanjoy Das 
736605f7795SMatthias Braun   for (auto &MO : MI->uses()) {
737605f7795SMatthias Braun     if (MO.isReg()) {
738605f7795SMatthias Braun       MachineOperand NewMO = MO;
739605f7795SMatthias Braun       if (MO.isUse()) {
740605f7795SMatthias Braun         NewMO.setIsKill(false);
741605f7795SMatthias Braun       } else {
742605f7795SMatthias Braun         assert(MO.isDef() && "Expected def or use");
743605f7795SMatthias Braun         NewMO.setIsDead(false);
744605f7795SMatthias Braun       }
745605f7795SMatthias Braun       MIB.add(NewMO);
746605f7795SMatthias Braun     } else {
747116bbab4SDiana Picus       MIB.add(MO);
748605f7795SMatthias Braun     }
749605f7795SMatthias Braun   }
75069fad079SSanjoy Das 
751c73c0307SChandler Carruth   MIB.setMemRefs(MI->memoperands());
75269fad079SSanjoy Das 
75369fad079SSanjoy Das   return MIB;
75469fad079SSanjoy Das }
75569fad079SSanjoy Das 
75669fad079SSanjoy Das /// Rewrite the null checks in NullCheckList into implicit null checks.
rewriteNullChecks(ArrayRef<ImplicitNullChecks::NullCheck> NullCheckList)75769fad079SSanjoy Das void ImplicitNullChecks::rewriteNullChecks(
75869fad079SSanjoy Das     ArrayRef<ImplicitNullChecks::NullCheck> NullCheckList) {
75969fad079SSanjoy Das   DebugLoc DL;
76069fad079SSanjoy Das 
761*9e6d1f4bSKazu Hirata   for (const auto &NC : NullCheckList) {
76269fad079SSanjoy Das     // Remove the conditional branch dependent on the null check.
7631b9fc8edSMatt Arsenault     unsigned BranchesRemoved = TII->removeBranch(*NC.getCheckBlock());
76469fad079SSanjoy Das     (void)BranchesRemoved;
76569fad079SSanjoy Das     assert(BranchesRemoved > 0 && "expected at least one branch!");
76669fad079SSanjoy Das 
767e57bf680SSanjoy Das     if (auto *DepMI = NC.getOnlyDependency()) {
768e57bf680SSanjoy Das       DepMI->removeFromParent();
769e57bf680SSanjoy Das       NC.getCheckBlock()->insert(NC.getCheckBlock()->end(), DepMI);
770e57bf680SSanjoy Das     }
771e57bf680SSanjoy Das 
7722f63cbccSSanjoy Das     // Insert a faulting instruction where the conditional branch was
7732f63cbccSSanjoy Das     // originally. We check earlier ensures that this bit of code motion
7742f63cbccSSanjoy Das     // is legal.  We do not touch the successors list for any basic block
7752f63cbccSSanjoy Das     // since we haven't changed control flow, we've just made it implicit.
7762f63cbccSSanjoy Das     MachineInstr *FaultingInstr = insertFaultingInstr(
777e173b9aeSSanjoy Das         NC.getMemOperation(), NC.getCheckBlock(), NC.getNullSucc());
77826dab3a4SQuentin Colombet     // Now the values defined by MemOperation, if any, are live-in of
77926dab3a4SQuentin Colombet     // the block of MemOperation.
7802f63cbccSSanjoy Das     // The original operation may define implicit-defs alongside
7812f63cbccSSanjoy Das     // the value.
782e173b9aeSSanjoy Das     MachineBasicBlock *MBB = NC.getMemOperation()->getParent();
7832f63cbccSSanjoy Das     for (const MachineOperand &MO : FaultingInstr->operands()) {
78426dab3a4SQuentin Colombet       if (!MO.isReg() || !MO.isDef())
78526dab3a4SQuentin Colombet         continue;
7860c476111SDaniel Sanders       Register Reg = MO.getReg();
78726dab3a4SQuentin Colombet       if (!Reg || MBB->isLiveIn(Reg))
78826dab3a4SQuentin Colombet         continue;
78912b69919SQuentin Colombet       MBB->addLiveIn(Reg);
79012b69919SQuentin Colombet     }
791e57bf680SSanjoy Das 
792e57bf680SSanjoy Das     if (auto *DepMI = NC.getOnlyDependency()) {
793e57bf680SSanjoy Das       for (auto &MO : DepMI->operands()) {
794f8c0cfc2SJonas Paulsson         if (!MO.isReg() || !MO.getReg() || !MO.isDef() || MO.isDead())
795e57bf680SSanjoy Das           continue;
796e57bf680SSanjoy Das         if (!NC.getNotNullSucc()->isLiveIn(MO.getReg()))
797e57bf680SSanjoy Das           NC.getNotNullSucc()->addLiveIn(MO.getReg());
798e57bf680SSanjoy Das       }
799e57bf680SSanjoy Das     }
800e57bf680SSanjoy Das 
801e173b9aeSSanjoy Das     NC.getMemOperation()->eraseFromParent();
802b04c181eSPhilip Reames     if (auto *CheckOp = NC.getCheckOperation())
803b04c181eSPhilip Reames       CheckOp->eraseFromParent();
80469fad079SSanjoy Das 
805b04c181eSPhilip Reames     // Insert an *unconditional* branch to not-null successor - we expect
806b04c181eSPhilip Reames     // block placement to remove fallthroughs later.
807e8e0f5caSMatt Arsenault     TII->insertBranch(*NC.getCheckBlock(), NC.getNotNullSucc(), nullptr,
808e173b9aeSSanjoy Das                       /*Cond=*/None, DL);
80969fad079SSanjoy Das 
8108ee6a30bSSanjoy Das     NumImplicitNullChecks++;
81169fad079SSanjoy Das   }
81269fad079SSanjoy Das }
81369fad079SSanjoy Das 
81469fad079SSanjoy Das char ImplicitNullChecks::ID = 0;
815900b6335SEugene Zelenko 
81669fad079SSanjoy Das char &llvm::ImplicitNullChecksID = ImplicitNullChecks::ID;
817900b6335SEugene Zelenko 
8181527baabSMatthias Braun INITIALIZE_PASS_BEGIN(ImplicitNullChecks, DEBUG_TYPE,
81969fad079SSanjoy Das                       "Implicit null checks", false, false)
820e57bf680SSanjoy Das INITIALIZE_PASS_DEPENDENCY(AAResultsWrapperPass)
8211527baabSMatthias Braun INITIALIZE_PASS_END(ImplicitNullChecks, DEBUG_TYPE,
82269fad079SSanjoy Das                     "Implicit null checks", false, false)
823