mirror of
https://github.com/llvm/llvm-project.git
synced 2025-04-29 08:26:07 +00:00
ae6a5c1d08
Just avoid crashing for now, we should be able to replace the blockaddresses themselves. BlockAddress::handleOperandChangeImpl assumes it can cast to Function. The verifier seems nonexistent and the langref isn't particularly explicit on what's allowed as a blockaddress operand. As far as I can tell bugpoint isn't doing anything to handle this. Something low level is broken with BlockAddress handling, demonstrated by reduce-functions-blockaddress-wrong-function.ll. The BasicBlock destructor of the deleted function is triggering replacement of blockaddresses for the kept function in some cases. I've only half debugged this but it seems like blockaddress is handled too-specially compared to other Constants. I have tentative patches to allow any constant to be a blockaddress input, but having the verifier check if it's really a function/block. https://reviews.llvm.org/D140909
31 lines
902 B
C++
31 lines
902 B
C++
//===- Utils.h - llvm-reduce utility functions ------------------*- C++ -*-===//
|
|
//
|
|
// 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 file contains some utility functions supporting llvm-reduce.
|
|
//
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
#ifndef LLVM_TOOLS_LLVM_REDUCE_DELTAS_UTILS_H
|
|
#define LLVM_TOOLS_LLVM_REDUCE_DELTAS_UTILS_H
|
|
|
|
#include "llvm/IR/Function.h"
|
|
#include "llvm/IR/Value.h"
|
|
#include "llvm/Support/CommandLine.h"
|
|
|
|
namespace llvm {
|
|
|
|
extern cl::opt<bool> Verbose;
|
|
|
|
Value *getDefaultValue(Type *T);
|
|
bool hasAliasUse(Function &F);
|
|
bool hasAliasOrBlockAddressUse(Function &F);
|
|
|
|
} // namespace llvm
|
|
|
|
#endif
|