[PATCH] D28829: [thumb, framelowering] Reset NoVRegs in Thumb1FrameLowering::emitPrologue.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 18 04:02:46 PST 2017


fhahn added a comment.

Thanks for having a look! Unfortunately I have no idea how to add a test. The problem is only exposed because ARMFrameLowering::adjustForSegmentedStacks verifies the machine function if expensive checks are enabled (which makes sure no virtual registers are present in functions marked as NoVRegs). The NoVRegs property is only violated inside the PrologEpilogInserter pass until doScavengeFrameVirtualRegs is called.

Cheers,
Florian


https://reviews.llvm.org/D28829





More information about the llvm-commits mailing list