[PATCH] D13922: Restrict tail duplication in the presence of subregisters

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 20 19:07:02 PDT 2015


MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.

ok, so this looks like a bug in the TailDuplication code, I wonder what it would take to actually fix TailDuplication rather than just disabling it. I guess disabling TailDuplication is still better than producing wrong code, so you can apply the patch if you must. But please state in the comment or somewhere that this aspect of the TailDuplication code is broken and should be fixed (or if possible try to fix it yourself so it produces the uses the correct subregister for the newly added phi operands).


Repository:
  rL LLVM

http://reviews.llvm.org/D13922





More information about the llvm-commits mailing list