[PATCH] D88494: Add "IncludeRemovable" parameter to TargetInstrInfo::DefinesPredicate

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 1 04:52:07 PDT 2020


samparker added a comment.

> Can we rename the function to something like "ClobbersPredicate", if that's what we actually care about here?

+1 from me. I also wonder renaming IncludesRemovable to something like Ignore/SkipDead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88494



More information about the llvm-commits mailing list