unittests: Actually test reverse iterators in Path tests
authorJustin Bogner <mail@justinbogner.com>
Wed, 16 Jul 2014 08:18:58 +0000 (08:18 +0000)
committerJustin Bogner <mail@justinbogner.com>
Wed, 16 Jul 2014 08:18:58 +0000 (08:18 +0000)
commit0c274aed23173518c0d17cc90592e980a4b4f2df
treef34c4a2dfbe80ea7b3cc4a99fbe13ee00f1c87b9
parentbf3d4301634240a29aadf5bd6be25b4acb08575d
unittests: Actually test reverse iterators in Path tests

This re-enables some #if 0'd code (since 2010) in the Path unittests
and makes at least a weak effort at testing sys::path's rbegin/rend.

This change was inspired by some test failures near uses of rbegin and
rend here:

    http://lab.llvm.org:8011/builders/clang-x86_64-linux-vg/builds/3209

The "valgrind was whining" comment looked promising in terms of a
simpler to debug case of the same errors. However, it appears that the
valgrind complaints the comment was referring to are distinct from the
ones in the frontend, since this updated test isn't complaining for me
under valgrind.

In any case, the disabled tests weren't helping anybody.

llvm-svn: 213125
llvm/unittests/Support/Path.cpp