[all-commits] [llvm/llvm-project] db091e: RISCV: Avoid GlobalISel build break in a future patch

Matt Arsenault via All-commits all-commits at lists.llvm.org
Mon Jul 13 11:03:45 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: db091e12b2358255d249c9f3f211f4474e2c723c
      https://github.com/llvm/llvm-project/commit/db091e12b2358255d249c9f3f211f4474e2c723c
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2020-07-13 (Mon, 13 Jul 2020)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVInstrInfoA.td
    M llvm/lib/Target/RISCV/RISCVInstructionSelector.cpp

  Log Message:
  -----------
  RISCV: Avoid GlobalISel build break in a future patch

The GlobalISelEmitter is stricter about matching timm instruction
outputs to timm inputs (although in an accidental sort of way that
doesn't hit a proper import failure error). Also, apparently no
intrinsic patterns were importing since the ID enum declaration was
missing.


  Commit: 2e2af6026b43511f8681397a9b6f13525dda970f
      https://github.com/llvm/llvm-project/commit/2e2af6026b43511f8681397a9b6f13525dda970f
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2020-07-13 (Mon, 13 Jul 2020)

  Changed paths:
    M llvm/test/TableGen/GlobalISelEmitter.td
    M llvm/test/TableGen/GlobalISelEmitterRegSequence.td
    M llvm/test/TableGen/GlobalISelEmitterSubreg.td
    M llvm/utils/TableGen/GlobalISelEmitter.cpp

  Log Message:
  -----------
  TableGen/GlobalISel: Emit enum names for reg class ID instead of value

This was emitting the raw value for the reg class ID with a comment
for the actual class name. Switch to emitting the qualified enum name
instead, which obviates the need for the comment and also helps keep
the lit tests on the emitter output more stable.


Compare: https://github.com/llvm/llvm-project/compare/2d3b8cc83fe8...2e2af6026b43


More information about the All-commits mailing list