On 12/14/2010 04:28 PM, Wesley Peck wrote:> On Dec 14, 2010, at 3:46 PM, Richard Pennington wrote: >> Notice that the label $BB0_1 is missing. If I disable filling in the >> branch delay slots, I get: > > Is this with the latest SVN HEAD version of LLVM or some other version? The delay slot filler and many other things have been updated for the Microblaze backend. In particular, the commit r120095 for the MBlaze backend fixed some issues with missing block labels due to AsmPrinter::isBlockOnlyReachableByFallthrough not taking into account delay slots in basic blocks. >[snip] Thanks Wesley! I'm currently using r119910. I'll update. -Rich
On 12/14/2010 04:32 PM, Richard Pennington wrote:> On 12/14/2010 04:28 PM, Wesley Peck wrote: >> On Dec 14, 2010, at 3:46 PM, Richard Pennington wrote: >>> Notice that the label $BB0_1 is missing. If I disable filling in the >>> branch delay slots, I get: >> >> Is this with the latest SVN HEAD version of LLVM or some other version? The delay slot filler and many other things have been updated for the Microblaze backend. In particular, the commit r120095 for the MBlaze backend fixed some issues with missing block labels due to AsmPrinter::isBlockOnlyReachableByFallthrough not taking into account delay slots in basic blocks. >> > [snip] > > Thanks Wesley! I'm currently using r119910. I'll update. > > -RichWesley, You changes work for me also. I applied something similar to the Mips CG and they worked there also. Thanks again. -Rich
Hi Richard,> You changes work for me also. I applied something similar to the Mips CG > and they worked there also. Thanks again.I can't reproduce the same problem here for Mips using clang, could you please attach the bitcode you used? Thanks -- Bruno Cardoso Lopes http://www.brunocardoso.cc