[clang] [llvm] [mlir] [polly] [NFC] Rename `Intrinsic::getDeclaration` to `getOrInsertDeclaration` (PR #111752)

Rahul Joshi via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 10 06:27:09 PDT 2024


jurahul wrote:

Yes, and that’s the motivation for this (this is a prep PR for that change)

On Thu, Oct 10, 2024 at 12:41 AM Matt Arsenault ***@***.***>
wrote:

> ***@***.**** approved this pull request.
>
> There are definitely places that would benefit from a getDeclaration.
> There are several places that have to awkwardly construct the intrinsic
> name to check getFunction
>
>> Reply to this email directly, view it on GitHub
> <https://github.com/llvm/llvm-project/pull/111752#pullrequestreview-2359359293>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/APRMUB7I5LGTJGKX6RR4NUTZ2YVUFAVCNFSM6AAAAABPVHZ3QGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNJZGM2TSMRZGM>
> .
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>


https://github.com/llvm/llvm-project/pull/111752


More information about the cfe-commits mailing list