[PATCH] D150388: [CodeGen]Allow targets to use target specific COPY instructions for live range splitting

Mon Jul 24 11:35:49 PDT 2023

vitalybuka added a comment.

In D150388#4529229 <https://reviews.llvm.org/D150388#4529229>, @arsenm wrote:

> In D150388#4528824 <https://reviews.llvm.org/D150388#4528824>, @vitalybuka wrote:
>
>> @yassingh @arsenm
>> The major target is broken for too long, we need either fix or revert, even if it's a chain of patches.
>
> I just posted about this issue:
> https://discourse.llvm.org/t/subreg-to-reg-semantics-or-x86s-zext-implementation-is-broken/72250
>
> Either x86 is broken or the definition of SUBREG_TO_REG is broken

So if we have not fix, just discussion, should we revert? Even if it's preexisted unknown bug, I don't think it's appropriate to keep regression in existing functionality, for sake of fixing previously broken or missing functionally. 
Is it possible to workaround/revert some change with "if (x86) do old way" ?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150388