From 3c148720faa8c4dbc76fde108a3e21f211c32d8a Mon Sep 17 00:00:00 2001 From: Daniel Neilson Date: Tue, 24 Apr 2018 21:12:45 +0000 Subject: [PATCH] [CaptureTracking] Fixup const correctness of DomTree arg (NFC) Summary: The PointerMayBeCapturedBefore function's DomTree arg should be const instead of non-const. There are no non-const uses of it in the function. llvm-svn: 330769 --- llvm/include/llvm/Analysis/CaptureTracking.h | 2 +- llvm/lib/Analysis/CaptureTracking.cpp | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/llvm/include/llvm/Analysis/CaptureTracking.h b/llvm/include/llvm/Analysis/CaptureTracking.h index 8d2c095d..7a869a5 100644 --- a/llvm/include/llvm/Analysis/CaptureTracking.h +++ b/llvm/include/llvm/Analysis/CaptureTracking.h @@ -46,7 +46,7 @@ namespace llvm { /// to speed up capture-tracker queries. bool PointerMayBeCapturedBefore(const Value *V, bool ReturnCaptures, bool StoreCaptures, const Instruction *I, - DominatorTree *DT, bool IncludeI = false, + const DominatorTree *DT, bool IncludeI = false, OrderedBasicBlock *OBB = nullptr); /// This callback is used in conjunction with PointerMayBeCaptured. In diff --git a/llvm/lib/Analysis/CaptureTracking.cpp b/llvm/lib/Analysis/CaptureTracking.cpp index 3b0026b..6566c7a 100644 --- a/llvm/lib/Analysis/CaptureTracking.cpp +++ b/llvm/lib/Analysis/CaptureTracking.cpp @@ -60,7 +60,7 @@ namespace { /// as the given instruction and the use. struct CapturesBefore : public CaptureTracker { - CapturesBefore(bool ReturnCaptures, const Instruction *I, DominatorTree *DT, + CapturesBefore(bool ReturnCaptures, const Instruction *I, const DominatorTree *DT, bool IncludeI, OrderedBasicBlock *IC) : OrderedBB(IC), BeforeHere(I), DT(DT), ReturnCaptures(ReturnCaptures), IncludeI(IncludeI), Captured(false) {} @@ -140,7 +140,7 @@ namespace { OrderedBasicBlock *OrderedBB; const Instruction *BeforeHere; - DominatorTree *DT; + const DominatorTree *DT; bool ReturnCaptures; bool IncludeI; @@ -184,7 +184,7 @@ bool llvm::PointerMayBeCaptured(const Value *V, /// queries about relative order among instructions in the same basic block. bool llvm::PointerMayBeCapturedBefore(const Value *V, bool ReturnCaptures, bool StoreCaptures, const Instruction *I, - DominatorTree *DT, bool IncludeI, + const DominatorTree *DT, bool IncludeI, OrderedBasicBlock *OBB) { assert(!isa(V) && "It doesn't make sense to ask whether a global is captured."); -- 2.7.4