[PATCH] D20295: When looking for a spill slot in reg scavenger, find one that matches RC

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Tue May 17 16:04:23 PDT 2016


kparzysz added inline comments.

================
Comment at: test/CodeGen/Hexagon/reg-scavenger-valid-slot.ll:106
@@ +105,3 @@
+  %niter.ncmp.7 = icmp eq i32 %niter.nsub.7, 0
+  br i1 %niter.ncmp.7, label %for.end.loopexit.unr-lcssa, label %for.body
+
----------------
kparzysz wrote:
> kparzysz wrote:
> > qcolombet wrote:
> > > Could you make the test smaller, e.g., via using inline asm to clobber registers?
> > > (You could also use mir input.)
> > Let me see what I can do with inline asm.  I haven't tried MIR with our backend so I'm not sure how much work it will be to get it to work.
> I need to keep enough registers live in a certain range.  Turns out that vector register inputs and outputs are not yet implemented for inline-asm.  I'll keep the code as is for now.
What I mean is that our backend crashes in isel-lowering on inline-asm with vector inputs/outputs.


Repository:
  rL LLVM

http://reviews.llvm.org/D20295





More information about the llvm-commits mailing list