<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Mar 15, 2018 at 11:10 AM David Blaikie via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">+Chandler who might have some thoughts on this.<br></div></blockquote><div><br></div><div>FWIW, I have no concerns about updating to a modern googletest. More modern the better IMO if someone is willing to do the work to make sure it works on all our platforms, etc.</div><div><br></div><div>However:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Could you provide an example here of the motivation for the feature you're missing?</div></blockquote><div><br></div><div>I definitely think that the whole space of combinatorial testing should be carefully discussed. Seems like a high risk of long-running tests w/o providing much incremental value.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"> Might help motivate the discussion (and/or we'll end up nitpicking how it could be done differently without that feature... - which is sort of where I'm going with this. Combinatorial test case expansion does seem a bit suspicious to me - I'd hope we could pick a few examples from the various equivalence classes & that would suffice?)</div><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Mar 12, 2018 at 9:01 AM James Henderson via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div>Hi all,<br><br></div>I'm currently writing some unit tests for some debug line error handling code I'm working on (see e.g. <a href="https://reviews.llvm.org/D44382" target="_blank">https://reviews.llvm.org/D44382</a>), and I just ran into an annoying disabled feature in gtest, specifically the "Combine" feature for use in combinatorially generating parameterised tests. A FIXME comment in ProfileData\CoverageMappingTest.cpp suggests that I'm not the only one to have tried and discovered that they cannot use this feature. The problem is that the version of googletest (v 1.8.0, released in Aug 2016) in the LLVM tree requires TR1 tuple support for this feature, which is not really supported in recent compilers, and has been explicitly disabled in our googletest CMakeLists.txt, thus disabling "Combine".<br><br></div>I did a bit of looking around, and v 1.8.0 is indeed the last officially tagged release of googletest. However, there has been a lot of development on the framework since that point, including a fix to enable use of Combine with std::tuple-supporting compilers. There have been a number of issue raised on the googletest issue tracker (see e.g. <a href="https://github.com/google/googletest/issues/1467" target="_blank">https://github.com/google/googletest/issues/1467</a> or <a href="https://github.com/google/googletest/issues/1079" target="_blank">https://github.com/google/googletest/issues/1079</a>) asking about a 1.9.0 release, and there has been zero response from anybody answering the query of when/if it will happen. In the meantime, the last release gets older and crustier...<br><br></div>I'd therefore like to propose something that might be seen as slightly controversial: update to use ToT googletest (or at least some reasonably recent version of master), at least until a new release is created.<br><br></div>Thoughts?<br><br></div>James<br></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>