[all-commits] [llvm/llvm-project] 147b94: [AArch64][GlobalISel] Split post-legalizer combine...

Jessica Paquette via All-commits all-commits at lists.llvm.org
Thu Oct 22 15:06:58 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 147b9497e79a98a8614b2b5eb4ba653b44f6b6f0
      https://github.com/llvm/llvm-project/commit/147b9497e79a98a8614b2b5eb4ba653b44f6b6f0
  Author: Jessica Paquette <jpaquette at apple.com>
  Date:   2020-10-22 (Thu, 22 Oct 2020)

  Changed paths:
    M llvm/lib/Target/AArch64/AArch64.h
    M llvm/lib/Target/AArch64/AArch64Combine.td
    M llvm/lib/Target/AArch64/AArch64TargetMachine.cpp
    M llvm/lib/Target/AArch64/CMakeLists.txt
    M llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp
    A llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerLowering.cpp
    M llvm/lib/Target/AArch64/GISel/AArch64PreLegalizerCombiner.cpp
    R llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-combiner-ext.mir
    R llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-combiner-rev.mir
    R llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-combiner-shuffle-splat.mir
    R llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-combiner-trn.mir
    R llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-combiner-uzp.mir
    R llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-combiner-vashr-vlshr.mir
    R llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-combiner-zip.mir
    A llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-lowering-ext.mir
    A llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-lowering-rev.mir
    A llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-lowering-shuffle-splat.mir
    A llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-lowering-trn.mir
    A llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-lowering-uzp.mir
    A llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-lowering-vashr-vlshr.mir
    A llvm/test/CodeGen/AArch64/GlobalISel/postlegalizer-lowering-zip.mir
    M llvm/test/CodeGen/AArch64/O0-pipeline.ll

  Log Message:
  -----------
  [AArch64][GlobalISel] Split post-legalizer combiner to allow for lowering at -O0

There are a lot of combines in AArch64PostLegalizerCombiner which exist to
facilitate instruction matching in the selector. (E.g. matching for G_ZIP and
other shuffle vector pseudos)

It still makes sense to select these instructions at -O0.

Matching earlier in a combiner can reduce complexity in the selector
significantly. For example, a good portion of our selection code for compares
would be a lot easier to represent in a combine.

This patch moves matching combines into a "AArch64PostLegalizerLowering"
combiner which runs at all optimization levels.

Also, while we're here, improve the documentation for the
AArch64PostLegalizerCombiner, and fix up the filepath in its file comment.

And also add a 'r' which somehow got dropped from a bunch of function names.

https://reviews.llvm.org/D89820




More information about the All-commits mailing list