search for: 086443

...ithout r81058 (at least on x86-64). I don't know if it has >> any other implications but it's definitely greatly stabilizing for >> our >> code base. > > Makes sense. > > Tanya, please pull http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20090831/086443.html > into 2.6 when convenient, thanks! > This can not go into 2.6, because r79742 is not in 2.6: http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20090817/085284.html Should this really be a release candidate? Its changing quite a bit and its not causing a regression. -Tan...

...uite a bit >> and its not causing a regression. > > No, the bigger patch should not go into 2.6. Devang, can you please > prepare a version of this patch that applies cleanly to the 2.6 > branch? > > http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20090831/086443.html hmmm... r81058 fixes a bug in the code that I added as part of r79742. We definitely do not want to add r79742 in 2.6. Are we sure that 4879 is not a recent regression ? - Devang

...t uses VLAs is horribly > broken without r81058 (at least on x86-64). I don't know if it has > any other implications but it's definitely greatly stabilizing for our > code base. Makes sense. Tanya, please pull http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20090831/086443.html into 2.6 when convenient, thanks! -Chris > > Thanks, > > Mike > > PS -- I am not subscribed to this list. > > -- > Michael P. Lyle > Chief Executive Officer > Translattice, Inc. > mlyle at translattice.com > > ___________________________________...

...release candidate? Its changing quite a bit > and its not causing a regression. No, the bigger patch should not go into 2.6. Devang, can you please prepare a version of this patch that applies cleanly to the 2.6 branch? http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20090831/086443.html -Chris -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20090908/3e6f5001/attachment.html>

...d its not causing a regression. >> >> No, the bigger patch should not go into 2.6. Devang, can you >> please prepare a version of this patch that applies cleanly to the >> 2.6 branch? >> >> http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20090831/086443.html > > hmmm... r81058 fixes a bug in the code that I added as part of > r79742. We definitely do not want to add r79742 in 2.6. Are we sure > that 4879 is not a recent regression ? Ok, if 2.6 is not affected, then we definitely don't want to mess with it. Thanks. -Chris

First-- thanks to Daniel Dunbar for reporting this issue from my earlier coarse report on IRC and to Devang Patel for fixing it. I'm writing to request that this fix (r81058) find its way into the 2.6 release. Code compiled with clang that uses VLAs is horribly broken without r81058 (at least on x86-64). I don't know if it has any other implications but it's definitely