[PATCH] D99173: Intrinsic::getName: require a Module argument

Jeroen Dobbelaere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 28 07:09:45 PDT 2021


jeroen.dobbelaere updated this revision to Diff 348521.
jeroen.dobbelaere edited the summary of this revision.
jeroen.dobbelaere added a comment.

- Rebased
- Made the M parameter for 'getName' mandatory (with an assertion check)
- I got mixed feedback with regard to the naming. I decided to settle on 'LLVMIntrinsicCopyOverloadedName2'.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99173/new/

https://reviews.llvm.org/D99173

Files:
  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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D99173.348521.patch
Type: text/x-patch
Size: 19691 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210528/4a3fd0d3/attachment.bin>


More information about the llvm-commits mailing list