Skip to content

Commit ea902d1

Browse files
committed
[IR] Remove deprecated opaque pointer migration methods
Remove the following methods: * Type::getNonOpaquePointerElementType() * Type::isOpaquePointerTy() * LLVMContext::supportsTypedPointers() * LLVMContext::setOpaquePointers() These were used temporarily during the opaque pointers migration, and are no longer needed.
1 parent d4f38f4 commit ea902d1

File tree

3 files changed

+0
-31
lines changed

3 files changed

+0
-31
lines changed

llvm/include/llvm/IR/LLVMContext.h

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -316,17 +316,6 @@ class LLVMContext {
316316
/// LLVMContext is used by compilation.
317317
void setOptPassGate(OptPassGate&);
318318

319-
/// Set whether opaque pointers are enabled. The method may be called multiple
320-
/// times, but only with the same value. Note that creating a pointer type or
321-
/// otherwise querying the opaque pointer mode performs an implicit set to
322-
/// the default value.
323-
[[deprecated("Opaque pointers are always enabled")]]
324-
void setOpaquePointers(bool Enable) const;
325-
326-
/// Whether typed pointers are supported. If false, all pointers are opaque.
327-
[[deprecated("Always returns false")]]
328-
bool supportsTypedPointers() const;
329-
330319
/// Get or set the current "default" target CPU (target-cpu function
331320
/// attribute). The intent is that compiler frontends will set this to a value
332321
/// that reflects the attribute that a function would get "by default" without

llvm/include/llvm/IR/Type.h

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -250,10 +250,6 @@ class Type {
250250
/// True if this is an instance of PointerType.
251251
bool isPointerTy() const { return getTypeID() == PointerTyID; }
252252

253-
/// True if this is an instance of an opaque PointerType.
254-
LLVM_DEPRECATED("Use isPointerTy() instead", "isPointerTy")
255-
bool isOpaquePointerTy() const { return isPointerTy(); };
256-
257253
/// Return true if this is a pointer type or a vector of pointer types.
258254
bool isPtrOrPtrVectorTy() const { return getScalarType()->isPointerTy(); }
259255

@@ -406,14 +402,6 @@ class Type {
406402

407403
inline StringRef getTargetExtName() const;
408404

409-
/// Only use this method in code that is not reachable with opaque pointers,
410-
/// or part of deprecated methods that will be removed as part of the opaque
411-
/// pointers transition.
412-
[[deprecated("Pointers no longer have element types")]]
413-
Type *getNonOpaquePointerElementType() const {
414-
llvm_unreachable("Pointers no longer have element types");
415-
}
416-
417405
/// Given vector type, change the element type,
418406
/// whilst keeping the old number of elements.
419407
/// For non-vectors simply returns \p EltTy.

llvm/lib/IR/LLVMContext.cpp

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -377,14 +377,6 @@ std::unique_ptr<DiagnosticHandler> LLVMContext::getDiagnosticHandler() {
377377
return std::move(pImpl->DiagHandler);
378378
}
379379

380-
void LLVMContext::setOpaquePointers(bool Enable) const {
381-
assert(Enable && "Cannot disable opaque pointers");
382-
}
383-
384-
bool LLVMContext::supportsTypedPointers() const {
385-
return false;
386-
}
387-
388380
StringRef LLVMContext::getDefaultTargetCPU() {
389381
return pImpl->DefaultTargetCPU;
390382
}

0 commit comments

Comments
 (0)