[all-commits] [llvm/llvm-project] b91f0d: [TableGen] Change backend callback to require cons...

[Rahul Joshi via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b91f0def6f261064cedfe0e62395c0097130b8f1
      https://github.com/llvm/llvm-project/commit/b91f0def6f261064cedfe0e62395c0097130b8f1
  Author: Rahul Joshi <rjoshi at nvidia.com>
  Date:   2024-10-04 (Fri, 04 Oct 2024)

  Changed paths:
    M llvm/include/llvm/TableGen/TableGenBackend.h
    M llvm/lib/TableGen/TableGenBackend.cpp

  Log Message:
  -----------
  [TableGen] Change backend callback to require const RecordKeeper (#111064)

Change TableGen backend callback function to require a const
RecordKeeper argument (by changing it from function_ref to just a
function pointer). This undoes parts of
https://github.com/llvm/llvm-project/pull/104716 which was added to
enable gradual migration of TableGen backends to use const RecordKeeper
(by allowing either const or non-const references). Now that all
backends have been migrated to const reference, we do not need this.

This is a part of effort to have better const correctness in TableGen
backends:


https://discourse.llvm.org/t/psa-planned-changes-to-tablegen-getallderiveddefinitions-api-potential-downstream-breakages/81089



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