[all-commits] [llvm/llvm-project] 754ff0: [TableGen][RISCV] Use getAllDerivedDefinitionsIfDe...

Alex Bradbury via All-commits all-commits at lists.llvm.org
Mon May 13 09:05:51 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 754ff0f54a4b09a8e4b00783475c51f66b949b66
      https://github.com/llvm/llvm-project/commit/754ff0f54a4b09a8e4b00783475c51f66b949b66
  Author: Alex Bradbury <asb at igalia.com>
  Date:   2024-05-13 (Mon, 13 May 2024)

  Changed paths:
    M llvm/utils/TableGen/RISCVTargetDefEmitter.cpp

  Log Message:
  -----------
  [TableGen][RISCV] Use getAllDerivedDefinitionsIfDefined in RISCVTargetDefEmitter (#91941)

getAllDerivedDefinitions produces a fatal error if there are no
definitions. In practice this isn't much of a problem for
llvm/lib/Target/RISCV/*.td where it's hard to imagine not having at
least one of the required defitions. But it limits our ability to
structure and maintain tests (which is how I came across this issue).

This commit moves to using getAllDerivedDefinitionsIfDefined and aims to
skip emission of data structures that make no sense if no definitions
were found.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list