You didn't update allocator.members\construct.pass.cpp in your patch.
On Sat, Mar 24, 2018 at 6:41 PM, Eric Fiselier <e...@efcs.ca> wrote: > Still? > > I would of hoped this patch fixed it. > > Could you provide the test output? > > /Eric > > On Sat, Mar 24, 2018 at 4:39 PM, Francois Pichet <pichet2...@gmail.com> > wrote: > >> Hi, >> >> I am seeing failure on this test too. >> test\std\utilities\memory\default.allocator\allocator.member >> s\construct.pass.cpp >> >> For the same reason : new optimized away. >> >> >> On Thu, Mar 22, 2018 at 5:28 PM, Eric Fiselier via cfe-commits < >> cfe-commits@lists.llvm.org> wrote: >> >>> Author: ericwf >>> Date: Thu Mar 22 14:28:09 2018 >>> New Revision: 328245 >>> >>> URL: http://llvm.org/viewvc/llvm-project?rev=328245&view=rev >>> Log: >>> Use DoNotOptimize to prevent new/delete elision. >>> >>> The new/delete tests, in particular those which test replacement >>> functions, often fail when the optimizer is enabled because the >>> calls to new/delete may be optimized away, regardless of their >>> side-effects. >>> >>> This patch converts the tests to use DoNotOptimize in order to prevent >>> the elision. >>> >>> >>> >> >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
