[PATCH] D150868: [CodeGen] Rename `MachineInstr::defs` to `MachineInstr::explicit_defs` (NFC)

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 19 01:05:23 PDT 2023


foad accepted this revision.
foad added a comment.
This revision is now accepted and ready to land.

This makes a lot of sense to me.

> It is more difficult to find an appropriate name for `uses`

Agreed. Maybe the solution is to remove `uses`? How "useful" is it? Can we migrate to `operands` and/or `implicit_operands`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150868/new/

https://reviews.llvm.org/D150868



More information about the llvm-commits mailing list