Woops. I misread the name of the test. I just committed a fix. Thanks for the heads up.
/Eric On Sat, Mar 24, 2018 at 4:44 PM, Francois Pichet <pichet2...@gmail.com> wrote: > 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
