[PATCH] D71132: PostRA Machine Sink should count COPY defining register used by another COPY

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 8 10:18:18 PST 2019


nhaehnle added a comment.

With this change, it looks as though accumulateUsedDefed is called multiple times on some paths. That seems wrong. It does seem plausible that we'd have to keep accumulating for all code paths.

Also, this needs a test.


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

https://reviews.llvm.org/D71132





More information about the llvm-commits mailing list