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