search for: computeliveinblock

...yself, and I think I have a working patch (attached). My own tests are all working wonderfully, and at fantastic performance! I'm looking forward to your patch to see whether we used the same approach or whether things could be improved further. Anyway, I've re-profiled the code and found ComputeLiveInBlocks to be the main hotspot now. Again it uses a linear search over the instructions so there's a chance this can be optimized as well... Kind regards, Nicolas -----Original Message----- From: David Greene [mailto:dag at cray.com] Sent: Wednesday, 24 September, 2008 18:07 To: Nicolas Capens Cc...

On Wednesday 24 September 2008 09:35, Nicolas Capens wrote: > Hi Dave, > > Did that patch of yours ever make it into trunk? I can't seem to find any > related checkin for PromoteMemoryToRegister.cpp. I've been doing some extra > profiling lately and the RewriteSingleStoreAlloca function alone is taking > a whopping 63% of execution time. I will commit it today along with

...ng > patch (attached). My own tests are all working wonderfully, and at > fantastic performance! > > I'm looking forward to your patch to see whether we used the same approach > or whether things could be improved further. > > Anyway, I've re-profiled the code and found ComputeLiveInBlocks to be the > main hotspot now. Again it uses a linear search over the instructions so > there's a chance this can be optimized as well... I don't quite understand how the logic of your patch is the same. In the original code, we're looking for a Load that uses the alloca before...

Hi Dave, Did that patch of yours ever make it into trunk? I can't seem to find any related checkin for PromoteMemoryToRegister.cpp. I've been doing some extra profiling lately and the RewriteSingleStoreAlloca function alone is taking a whopping 63% of execution time. Thanks! Nicolas -----Original Message----- From: llvmdev-bounces at cs.uiuc.edu [mailto:llvmdev-bounces at cs.uiuc.edu]