[PATCH] D36386: [clang] Remove unit test which uses reverse-iterate flag

2017-08-24 Thread Mandeep Singh Grang via Phabricator via cfe-commits
mgrang added a comment. In https://reviews.llvm.org/D36386#851853, @dblaikie wrote: > sounds good - so long as other tests would fail if the fix this test was > testing wasn't present (on a reverse iteration enabled build) Thanks! Yes, I verified that the test

[PATCH] D36386: [clang] Remove unit test which uses reverse-iterate flag

2017-08-24 Thread Mandeep Singh Grang via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL311720: [clang] Remove unit test which uses reverse-iterate flag (authored by mgrang). Changed prior to commit: https://reviews.llvm.org/D36386?vs=111557=112624#toc Repository: rL LLVM

[PATCH] D36386: [clang] Remove unit test which uses reverse-iterate flag

2017-08-24 Thread David Blaikie via Phabricator via cfe-commits
dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. sounds good - so long as other tests would fail if the fix this test was testing wasn't present (on a reverse iteration enabled build) https://reviews.llvm.org/D36386

[PATCH] D36386: [clang] Remove unit test which uses reverse-iterate flag

2017-08-17 Thread Mandeep Singh Grang via Phabricator via cfe-commits
mgrang updated this revision to Diff 111557. mgrang retitled this revision from "[clang] Remove unit test which uses reverse-iterate and fix a PointerLikeTypeTrait specialization" to "[clang] Remove unit test which uses reverse-iterate flag". mgrang edited the summary of this revision.