[PATCH] D153164: [AArch64][GlobalISel] Select G_UADDE/G_SADDE/G_USUBE/G_SSUBE
Tobias Stadler via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Jun 17 10:41:22 PDT 2023
tobias-stadler added a comment.
In D153164#4430170 <https://reviews.llvm.org/D153164#4430170>, @tschuett wrote:
> How did you noticed the fallback?
> They were marked as legal, but there was no support in instruction selection?
We noticed this, because our application (database query compilation) makes frequent use of 128bit add/sub overflow intrinsics, which produced a significant amount of fallbacks when using GlobalISel. With this patch, we don't fallback anymore (at least from what we have tested so far).
The G_*E operations were marked as legal by https://reviews.llvm.org/D95325, but there was no selector implementation provided at that time.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153164/new/
https://reviews.llvm.org/D153164
More information about the llvm-commits
mailing list