[all-commits] [llvm/llvm-project] 187686: [GlobalISel] LegalizationArtifactCombiner: Fix a b...

Volkan Keles via All-commits all-commits at lists.llvm.org
Fri Feb 14 10:46:13 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 187686a22f97662c981b9bffb998b6010ae1c401
      https://github.com/llvm/llvm-project/commit/187686a22f97662c981b9bffb998b6010ae1c401
  Author: Volkan Keles <vkeles at apple.com>
  Date:   2020-02-14 (Fri, 14 Feb 2020)

  Changed paths:
    M llvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h
    M llvm/include/llvm/CodeGen/GlobalISel/Utils.h
    M llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp
    M llvm/lib/CodeGen/GlobalISel/Utils.cpp
    M llvm/lib/Target/Mips/MipsRegisterBankInfo.cpp
    A llvm/test/CodeGen/AArch64/GlobalISel/artifact-combine-unmerge.mir

  Log Message:
  -----------
  [GlobalISel] LegalizationArtifactCombiner: Fix a bug in tryCombineMerges

Like COPY instructions explained in D70616, we don't check the constraints
when combining G_UNMERGE_VALUES. Use the same logic used in D70616 to check
if registers can be replaced, or a COPY instruction needs to be built.

https://reviews.llvm.org/D70564




More information about the All-commits mailing list