[llvm-dev] LiveInterval error with 2 dead defs
Krzysztof Parzyszek via llvm-dev
llvm-dev at lists.llvm.org
Tue Oct 8 14:24:01 PDT 2019
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.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev