<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Aug 9, 2019, at 04:13, Eric Fiselier <<a href="mailto:eric@efcs.ca" class="">eric@efcs.ca</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div class="">If I understand this correctly, the headers now require CMake build and configuration steps before</div><div class="">they've valid?</div><div class=""><br class=""></div><div class="">Code should be valid at rest -- without any intervention, configuration, setup, or build steps.</div></div></div></blockquote><div><br class=""></div><div>I strongly disagree. Our libraries inherently need to be configured, and having defaults that appear to just work (but aren't necessarily what you want) is harmful. We have plenty of those in libc++ (like that ABI library you use or whether per-TU insulation is provided) and I won't repeat that mistake in pstl. If you want convenience (which is a 100% fine goal), having a pre-populated CMake cache for common configurations is the way to go.</div><div><br class=""></div><div>Louis</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class=""> </div><div class="">/Eric<br class=""></div><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><br class=""></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 8, 2019 at 8:42 AM Louis Dionne via libcxx-commits <<a href="mailto:libcxx-commits@lists.llvm.org" class="">libcxx-commits@lists.llvm.org</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Author: ldionne<br class="">
Date: Thu Aug 8 05:43:13 2019<br class="">
New Revision: 368285<br class="">
<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=368285&view=rev" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-project?rev=368285&view=rev</a><br class="">
Log:<br class="">
[pstl] Error out when the backend is left unspecified<br class="">
<br class="">
Instead of silently falling back to the serial backend, it's better if<br class="">
we fail loudly when the parallel backend is left unspecified. Since we<br class="">
have a __pstl_config_site header, a backend should always be specified.<br class="">
<br class="">
Modified:<br class="">
pstl/trunk/include/pstl/internal/pstl_config.h<br class="">
<br class="">
Modified: pstl/trunk/include/pstl/internal/pstl_config.h<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project/pstl/trunk/include/pstl/internal/pstl_config.h?rev=368285&r1=368284&r2=368285&view=diff" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-project/pstl/trunk/include/pstl/internal/pstl_config.h?rev=368285&r1=368284&r2=368285&view=diff</a><br class="">
==============================================================================<br class="">
--- pstl/trunk/include/pstl/internal/pstl_config.h (original)<br class="">
+++ pstl/trunk/include/pstl/internal/pstl_config.h Thu Aug 8 05:43:13 2019<br class="">
@@ -19,9 +19,7 @@<br class="">
#define _PSTL_VERSION_PATCH (_PSTL_VERSION % 10)<br class="">
<br class="">
#if !defined(_PSTL_PAR_BACKEND_SERIAL) && !defined(_PSTL_PAR_BACKEND_TBB)<br class="">
-// TODO: In the future, we need to handle this setting using a configure-time<br class="">
-// option and something like a __config_site header.<br class="">
-# define _PSTL_PAR_BACKEND_SERIAL<br class="">
+# error "A parallel backend must be specified"<br class="">
#endif<br class="">
<br class="">
// Check the user-defined macro for warnings<br class="">
<br class="">
<br class="">
_______________________________________________<br class="">
libcxx-commits mailing list<br class="">
<a href="mailto:libcxx-commits@lists.llvm.org" target="_blank" class="">libcxx-commits@lists.llvm.org</a><br class="">
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/libcxx-commits" rel="noreferrer" target="_blank" class="">https://lists.llvm.org/cgi-bin/mailman/listinfo/libcxx-commits</a><br class="">
</blockquote></div></div>
</div></blockquote></div><br class=""></body></html>