Re: r329342 - PR36992: do not store beyond the dsize of a class object unless we know

2018-04-06 Thread Richard Smith via cfe-commits
Thanks, I see the problem. On Fri, 6 Apr 2018, 11:56 via cfe-commits, wrote: > Hi Richard, > > I don't know if you are aware, but the test you added in this commit, > tail-padding.cpp is currently failing on one of the ARM bots. Can you take > a look? > > >

RE: r329342 - PR36992: do not store beyond the dsize of a class object unless we know

2018-04-06 Thread via cfe-commits
Hi Richard, I don't know if you are aware, but the test you added in this commit, tail-padding.cpp is currently failing on one of the ARM bots. Can you take a look? http://lab.llvm.org:8011/builders/clang-cmake-armv7-selfhost-neon/builds/178 TEST 'Clang ::

r329342 - PR36992: do not store beyond the dsize of a class object unless we know

2018-04-05 Thread Richard Smith via cfe-commits
Author: rsmith Date: Thu Apr 5 13:52:58 2018 New Revision: 329342 URL: http://llvm.org/viewvc/llvm-project?rev=329342=rev Log: PR36992: do not store beyond the dsize of a class object unless we know the tail padding is not reused. We track on the AggValueSlot (and through a couple of other