Improve fix to #317 to restore ability to handle incomplete types

Author: hsutter

include/cpp2util.h

@@ -522,12 +522,11 @@ template<typename T>
 template<typename T>
 constexpr bool prefer_pass_by_value =
     sizeof(T) <= 2*sizeof(void*)
-    && std::is_trivially_copy_constructible_v<T>
-    && !std::is_class_v<T>
-    && !std::is_union_v<T>
-    && !std::is_array_v<T>
-    && !std::is_function_v<T>
-    ;
+    && std::is_trivially_copy_constructible_v<T>;
+
+template<typename T>
+    requires std::is_class_v<T> || std::is_union_v<T> || std::is_array_v<T> || std::is_function_v<T>
+constexpr bool prefer_pass_by_value<T> = false;
 
 template<typename T>
     requires (!std::is_void_v<T>)

regression-tests/test-results/version

@@ -1,5 +1,5 @@
 
-cppfront compiler v0.2.1   Build 8529:1436
+cppfront compiler v0.2.1   Build 8529:1625
 Copyright(c) Herb Sutter   All rights reserved
 
 SPDX-License-Identifier: CC-BY-NC-ND-4.0

source/build.info

@@ -1 +1 @@
-"8529:1436"
+"8529:1625"