llvm/llvm-project

[AArch64][GlobalISel] ABD operations and combines

Open

#118 085 ouverte le 29 nov. 2024

Voir sur GitHub
 (17 commentaires) (0 réactions) (0 assignés)C++ (10 782 forks)batch import
backend:AArch64good first issuellvm:globaliselmissed-optimization

Métriques du dépôt

Stars
 (26 378 stars)
Métriques de merge PR
 (Merge moyen 1j 2h) (1 000 PRs mergées en 30 j)

Description

SDAG, via these nodes can pattern-recognise and optimise abd nodes. We should add the same operations for global isel.

  /// ABDS/ABDU - Absolute difference - Return the absolute difference between
  /// two numbers interpreted as signed/unsigned.
  /// i.e trunc(abs(sext(Op0) - sext(Op1))) becomes abds(Op0, Op1)
  ///  or trunc(abs(zext(Op0) - zext(Op1))) becomes abdu(Op0, Op1)
  ABDS,
  ABDU,

There are tests in, for example, llvm/test/CodeGen/AArch64/abd-combine.ll. https://godbolt.org/z/71PGjf486

Guide contributeur

Direction de recherche
Étudiez l'implémentation SDAG existante des nœuds ABDS et ABDU dans LLVM, en particulier dans lib/CodeGen/SelectionDAG/ et les motifs pertinents dans lib/Target/AArch64/. Ensuite, implémentez les opérations génériques correspondantes dans GlobalISel (probablement dans lib/CodeGen/GlobalISel/). Utilisez le fichier de test existant llvm/test/CodeGen/AArch64/abd combine.ll comme guide pour vérifier la correction. Examinez les commentaires et la discussion dans l'issue pour des conseils supplémentaires.
Stack technique
cpp
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++LLVMcompiler fundamentals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40