search

Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

Fix and consolidate the test allocators. #28686

Open Quuxplusone opened 8 years ago

Quuxplusone commented 8 years ago
Bugzilla Link PR28687
Status NEW
Importance P normal
Reported by Eric Fiselier (eric@efcs.ca)
Reported on 2016-07-24 19:12:57 -0700
Last modified on 2018-06-29 21:25:06 -0700
Version unspecified
Hardware PC Windows NT
CC llvm-bugs@lists.llvm.org, mclow.lists@gmail.com
Fixed by commit(s)
Attachments
Blocks
Blocked by
See also

There are a bunch of problems in our test suite relating to our test allocator classes. First they have separate duplicate definitions in a bunch of different test files. We should pull them all out into a single support header.

This will help solve the second problem, which is many of these allocator don't conform to the standard requirements and therefore it makes the tests non portable.

Doing this will be a lot of work. This bug tracks the status of that work.

Quuxplusone commented 8 years ago 
I remember some work on this at the Bloomberg sprint.
https://reviews.llvm.org/D16967 is a start.
Quuxplusone commented 8 years ago

And https://reviews.llvm.org/D16970, too.

Quuxplusone commented 6 years ago

I did some of this in test/std/strings as part of D48616.