[llvm-dev] LiveInterval error with 2 dead defs

[Krzysztof Parzyszek via llvm-dev]

I have an alternative at https://reviews.llvm.org/D68666.

--
Krzysztof Parzyszek  kparzysz at quicinc.com<mailto:kparzysz at quicinc.com>   AI tools development

From: Matt Arsenault <whatmannerofburgeristhis at gmail.com> On Behalf Of Matt Arsenault
Sent: Monday, October 7, 2019 4:50 PM
To: Krzysztof Parzyszek <kparzysz at quicinc.com>
Cc: Quentin Colombet <qcolombet at apple.com>; Matthew P Arsenault <Matthew.Arsenault at amd.com>; llvm-dev at lists.llvm.org; Pranav Bhandarkar <pranavb at quicinc.com>
Subject: [EXT] Re: [llvm-dev] LiveInterval error with 2 dead defs




On Oct 7, 2019, at 14:04, Krzysztof Parzyszek via llvm-dev <llvm-dev at lists.llvm.org<mailto:llvm-dev at lists.llvm.org>> wrote:

The associated patch caused a compilation problems on Hexagon: https://bugs.llvm.org/show_bug.cgi?id=43302

The splitting of a live interval should not be done automatically upon creation.  Calling LIS->getInterval(Reg) should not go around changing the code behind the scenes.

There is already a function “splitSeparateComponents” that does that.  It should be added where it’s missing.


The patch is using splitSeparateComponents. Where else would it go? The problem is with the initial live interval creation.

-Matt
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20191008/85707ac8/attachment.html>