[PATCH] D155063: [RISCV] Simplify operand logic in vmerge combine. NFCI

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 13 11:32:11 PDT 2023


reames added a comment.

After staring at this for a while last night and not quite being able to convince myself of the correctness, I went through and split out pieces one at a time this morning - mostly as an exercise in establishing all the intermediate lemmas and justifying it to myself.  I think I've landed all the major pieces of this.  Can you do one last rebase to see if I missed anything you consider important?

p.s. Thank you for posting it.  I definitely agree that the overall result is a lot cleaner.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155063



More information about the llvm-commits mailing list