search for: spillobjects

...est effort" method. It is not guaranteed to return true for all instructions of the form, it is just a hint. A couple of the methods you're adding to MachineFrameInfo.h are large, they should be moved to the .cpp file, allowing you to avoid <limits> in the header. + /// SpillObjects - A set of frame indices represnting spill slots. typo represnting. Why does SpillObjects need to be a DenseSet? It seems that it is created in order. I think it can just be a vector which is looked up with a binary search. Instead of making CreateStackObject take a "bool isSpill&qu...

...s not guaranteed to return true for all instructions of the form, it > is just a hint. Right. > A couple of the methods you're adding to MachineFrameInfo.h are large, > they should be moved to the .cpp file, allowing you to avoid <limits> > in the header. Ok. > Why does SpillObjects need to be a DenseSet? It seems that it is > created in order. I think it can just be a vector which is looked up > with a binary search. I wasn't sure ordering was guaranteed. As I noted to Dan, I'd like to get rid of this entirely. > Instead of making CreateStackObject take...

Attached is a patch to print asm comments for spill information. We've discussed the mechanisms before but I wanted to run the patch by everyone before I start to commit pieces. -Dave -------------- next part -------------- A non-text attachment was scrubbed... Name: spillcomments.patch Type: text/x-diff Size: 58930 bytes Desc: not available URL: