6b62a9135a
I'd reverted this in 6c7805d5d1 after a bad stage. Original commit messsage follows: [NFC][RemoveDIs] Bulk update utilities to insert with iterators As part of the RemoveDIs project we need LLVM to insert instructions using iterators wherever possible, so that the iterators can carry a bit of debug-info. This commit implements some of that by updating the contents of llvm/lib/Transforms/Utils to always use iterator-versions of instruction constructors. There are two general flavours of update: * Almost all call-sites just call getIterator on an instruction * Several make use of an existing iterator (scenarios where the code is actually significant for debug-info) The underlying logic is that any call to getFirstInsertionPt or similar APIs that identify the start of a block need to have that iterator passed directly to the insertion function, without being converted to a bare Instruction pointer along the way. I've also switched DemotePHIToStack to take an optional iterator: it needs to take an iterator, and having a no-insert-location behaviour appears to be important. The constructors for ICmpInst and FCmpInst have been updated too. They're the only instructions that take block _references_ rather than pointers for certain calls, and a future patch is going to make use of default-null block insertion locations. All of this should be NFC.
67 lines
2.6 KiB
C++
67 lines
2.6 KiB
C++
//===- StripGCRelocates.cpp - Remove gc.relocates inserted by RewriteStatePoints===//
|
|
//
|
|
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
|
|
// See https://llvm.org/LICENSE.txt for license information.
|
|
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
|
|
//
|
|
//===----------------------------------------------------------------------===//
|
|
//
|
|
// This is a little utility pass that removes the gc.relocates inserted by
|
|
// RewriteStatepointsForGC. Note that the generated IR is incorrect,
|
|
// but this is useful as a single pass in itself, for analysis of IR, without
|
|
// the GC.relocates. The statepoint and gc.result intrinsics would still be
|
|
// present.
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
#include "llvm/Transforms/Utils/StripGCRelocates.h"
|
|
#include "llvm/IR/Function.h"
|
|
#include "llvm/IR/InstIterator.h"
|
|
#include "llvm/IR/Instructions.h"
|
|
#include "llvm/IR/Statepoint.h"
|
|
|
|
using namespace llvm;
|
|
|
|
static bool stripGCRelocates(Function &F) {
|
|
// Nothing to do for declarations.
|
|
if (F.isDeclaration())
|
|
return false;
|
|
SmallVector<GCRelocateInst *, 20> GCRelocates;
|
|
// TODO: We currently do not handle gc.relocates that are in landing pads,
|
|
// i.e. not bound to a single statepoint token.
|
|
for (Instruction &I : instructions(F)) {
|
|
if (auto *GCR = dyn_cast<GCRelocateInst>(&I))
|
|
if (isa<GCStatepointInst>(GCR->getOperand(0)))
|
|
GCRelocates.push_back(GCR);
|
|
}
|
|
// All gc.relocates are bound to a single statepoint token. The order of
|
|
// visiting gc.relocates for deletion does not matter.
|
|
for (GCRelocateInst *GCRel : GCRelocates) {
|
|
Value *OrigPtr = GCRel->getDerivedPtr();
|
|
Value *ReplaceGCRel = OrigPtr;
|
|
|
|
// All gc_relocates are i8 addrspace(1)* typed, we need a bitcast from i8
|
|
// addrspace(1)* to the type of the OrigPtr, if the are not the same.
|
|
if (GCRel->getType() != OrigPtr->getType())
|
|
ReplaceGCRel = new BitCastInst(OrigPtr, GCRel->getType(), "cast", GCRel->getIterator());
|
|
|
|
// Replace all uses of gc.relocate and delete the gc.relocate
|
|
// There maybe unncessary bitcasts back to the OrigPtr type, an instcombine
|
|
// pass would clear this up.
|
|
GCRel->replaceAllUsesWith(ReplaceGCRel);
|
|
GCRel->eraseFromParent();
|
|
}
|
|
return !GCRelocates.empty();
|
|
}
|
|
|
|
PreservedAnalyses StripGCRelocates::run(Function &F,
|
|
FunctionAnalysisManager &AM) {
|
|
if (!stripGCRelocates(F))
|
|
return PreservedAnalyses::all();
|
|
|
|
// Removing gc.relocate preserves the CFG, but most other analysis probably
|
|
// need to re-run.
|
|
PreservedAnalyses PA;
|
|
PA.preserveSet<CFGAnalyses>();
|
|
return PA;
|
|
}
|