[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