[PATCH] D110829: [X86] Copy registers in reverse order in convertToThreeAddress

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 30 06:41:16 PDT 2021


foad added reviewers: RKSimon, craig.topper.
foad added a comment.

1. I don't know if it's safe to rely on the kill flag being on the //first// operand that uses a particular reg. If not I'll rework the patch.
2. I don't know how to test this without removing the `false` from `addPass(&TwoAddressInstructionPassID, false)` in TargetPassConfig, which will currently still fail for a few other reasons.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110829



More information about the llvm-commits mailing list