Intrinsic::getName: require a Module argument
authorJeroen Dobbelaere <jeroen.dobbelaere@synopsys.com>
Mon, 14 Jun 2021 12:52:29 +0000 (14:52 +0200)
committerJeroen Dobbelaere <jeroen.dobbelaere@synopsys.com>
Mon, 14 Jun 2021 12:52:29 +0000 (14:52 +0200)
commitbb8ce25e88218be60d2a4ea9c9b0b721809eff27
treee72f60d66fe0fce564a53648aae5dae7b9340079
parent9f967eed89e66e39909c59ec0246dc2877d75f51
Intrinsic::getName: require a Module argument

Ensure that we provide a `Module` when checking if a rename of an intrinsic is necessary.

This fixes the issue that was detected by https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=32288
(as mentioned by @fhahn), after committing D91250.

Note that the `LLVMIntrinsicCopyOverloadedName` is being deprecated in favor of `LLVMIntrinsicCopyOverloadedName2`.

Reviewed By: nikic

Differential Revision: https://reviews.llvm.org/D99173
12 files changed:
llvm/docs/ReleaseNotes.rst
llvm/include/llvm-c/Core.h
llvm/include/llvm/IR/Intrinsics.h
llvm/lib/CodeGen/MachineOperand.cpp
llvm/lib/CodeGen/ReplaceWithVeclib.cpp
llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp
llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp
llvm/lib/IR/AutoUpgrade.cpp
llvm/lib/IR/Core.cpp
llvm/lib/IR/Function.cpp
llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp
llvm/test/Assembler/auto_upgrade_intrinsics.ll