search for: readsregist

...terator I = MBB->begin(); I != MBB->end (); ++I) { + for (unsigned i = 0, e = CSI.size(); i != e; ++i) { + unsigned Reg = CSI[i].getReg(); + // If instruction I reads or modifies Reg, add it to UsedCSRegs, + // CSRUsed map for the current block. + if (I->readsRegister(Reg, TRI) || I->modifiesRegister(Reg, TRI)) { readsRegister and modifiesRegister both scan the operands. It's probably better to manually walk through the operands. 9. + // If all uses of CSRegs are in the entry block, there is nothing + // to shrink wrap: spills go in entry block,...

Here is an updated patch for shrink wrapping with: - spills/restores done with stack slot stores/loads - stack adjustment removed - refactoring (but still in need of more) - spill/restore insertion code unified with spill/restore placement code Documentation available here<http://wiki.github.com/jdmdj/llvm-work/shrink-wrapping-work> illustrates shrink wrapping with loops and discusses a

On Mon, Mar 2, 2009 at 10:35 AM, Evan Cheng <echeng at apple.com> wrote: > > On Mar 1, 2009, at 2:57 PM, John Mosby wrote: > > Obviously, all of this applies only when spills are done with push/pop, > which is the case on x86. I used this issue to start looking at generalizing > how spills and restores are handled, before looking too closely at other > targets, and

...+ for (unsigned i = 0, e = CSI.size(); i != e; ++i) { > > > + unsigned Reg = CSI[i].getReg(); > > > + // If instruction I reads or modifies Reg, add it to UsedCSRegs, > > > + // CSRUsed map for the current block. > > > + if (I->readsRegister(Reg, TRI) || I->modifiesRegister(Reg, TRI)) { > > readsRegister and modifiesRegister both scan the operands. It's probably > better to manually walk through the operands. > Ok, that helps. I read the code but could not decide which way to do it at first. > 9. > + // I...

...e; ++i) { >> >> >> +        unsigned Reg = CSI[i].getReg(); >> >> >> +        // If instruction I reads or modifies Reg, add it to UsedCSRegs, >> >> >> +        // CSRUsed map for the current block. >> >> >> +        if (I->readsRegister(Reg, TRI) || I->modifiesRegister(Reg, TRI)) >> { >> readsRegister and modifiesRegister both scan the operands. It's probably >> better to manually walk through the operands. > > Ok, that helps. I read the code but could not decide which way to do it at > first. &...