[all-commits] [llvm/llvm-project] 6eeac6: GlobalISel: Fix double printing new instructions i...

Matt Arsenault via All-commits all-commits at lists.llvm.org
Tue Jun 9 12:03:11 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 6eeac6ae33046f022f2d2c857ef38d2329acfc88
      https://github.com/llvm/llvm-project/commit/6eeac6ae33046f022f2d2c857ef38d2329acfc88
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2020-06-09 (Tue, 09 Jun 2020)

  Changed paths:
    M llvm/lib/CodeGen/GlobalISel/Legalizer.cpp

  Log Message:
  -----------
  GlobalISel: Fix double printing new instructions in legalizer

New instructions were getting printed both in createdInstr, and in the
final printNewInstrs, so it made it look like the same instructions
were created twice. This overall made reading the debug output
harder. Stop printing the initial construction and only print new
instructions in the summary at the end. This avoids printing the less
useful case where instructions are sometimes initially created with no
operands.

I'm not sure this is the correct instance to remove; now the visible
ordering is different. Now you will typically see the one erased
instruction message before all the new instructions in order. I think
this is the more logical view of typical legalization changes,
although it's mechanically backwards from the normal
insert-new-erase-old pattern.




More information about the All-commits mailing list