[PATCH] D31908: [AntiDepBreaker] Don't rename callee saved register restore instruction

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 17:11:02 PDT 2017


MatzeB added a comment.

As a possible better design: Last time I had a similar problem in the context of copy propagation I checked whether the uses/defs involved had a register forced in their MCInstrDesc description and I assumed all implicit uses/defs to be constrained as well as they are typically used for calls/asm.

That seems to work for most targets (some had additional constraints not matched by this heuristic but I think the big ones X86, ARM, AArch64, Power lit tests worked when I experimented with that) though none of these heuristics made it into trunk so take this with a grain of salt...


https://reviews.llvm.org/D31908





More information about the llvm-commits mailing list