[PATCH] D63510: [LiveInterval] Removed bogus empty subrange assert

Tim Renouf via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 19 02:30:46 PDT 2019


tpr marked an inline comment as done.
tpr added inline comments.


================
Comment at: test/CodeGen/AMDGPU/empty-subrange.mir:1
+# RUN: llc -mtriple=amdgcn -mcpu=gfx802 -verify-machineinstrs -start-after amdgpu-isel -o - %s | FileCheck -check-prefix=GCN %s
+#
----------------
arsenm wrote:
> Should be restricted to a narrow range of passes in regalloc
I tried capturing mir just before the live intervals before regalloc, which is the one where it fell over, and running from there, but it did not reproduce the failure. I think it is too sensitive to what order it sees the MOs when iterating, which would be potentially different between reading a mir file and getting real output from earlier passes.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63510





More information about the llvm-commits mailing list