Replaced version (unittest) with version (DRuntimeUnittest) to avoid unnecessary overhead when compiling with -unittest

src/core/atomic.d

@@ -819,7 +819,7 @@ private
 ////////////////////////////////////////////////////////////////////////////////
 
 
-version (unittest)
+version (CoreUnittest)
 {
     void testXCHG(T)(T val) pure nothrow @nogc @trusted
     in

src/core/internal/array/operations.d

@@ -449,12 +449,12 @@ bool contains(T)(const scope T[] ary, const scope T[] vals...)
 
 // tests
 
-version (unittest) template TT(T...)
+version (CoreUnittest) template TT(T...)
 {
     alias TT = T;
 }
 
-version (unittest) template _arrayOp(Args...)
+version (CoreUnittest) template _arrayOp(Args...)
 {
     alias _arrayOp = arrayOp!Args;
 }

src/core/internal/parseoptions.d

@@ -78,7 +78,7 @@ bool parseOptions(CFG)(ref CFG cfg, string opt)
         static if (is(typeof(__traits(getMember, CFG, "help"))))
             if (name == "help")
             {
-                version (unittest) {} else
+                version (CoreUnittest) {} else
                 cfg.help();
                 opt = skip!isspace(tail);
                 continue;
@@ -135,7 +135,7 @@ private:
 
 bool optError(const scope char[] msg, const scope char[] name, const(char)[] errName)
 {
-    version (unittest) if (inUnittest) return false;
+    version (CoreUnittest) if (inUnittest) return false;
 
     fprintf(stderr, "%.*s %.*s option '%.*s'.\n",
             cast(int)msg.length, msg.ptr,
@@ -242,7 +242,7 @@ do
 
 bool parseError(const scope char[] exp, const scope char[] opt, const scope char[] got, const(char)[] errName)
 {
-    version (unittest) if (inUnittest) return false;
+    version (CoreUnittest) if (inUnittest) return false;
 
     fprintf(stderr, "Expecting %.*s as argument for %.*s option '%.*s', got '%.*s' instead.\n",
             cast(int)exp.length, exp.ptr,
@@ -254,7 +254,7 @@ bool parseError(const scope char[] exp, const scope char[] opt, const scope char
 
 size_t min(size_t a, size_t b) { return a <= b ? a : b; }
 
-version (unittest) __gshared bool inUnittest;
+version (CoreUnittest) __gshared bool inUnittest;
 
 unittest
 {

src/core/internal/traits.d

@@ -157,6 +157,7 @@ template staticIota(int beg, int end)
 }
 
 private struct __InoutWorkaroundStruct {}
+@property T rvalueOf(T)(T val) { return val; }
 @property T rvalueOf(T)(inout __InoutWorkaroundStruct = __InoutWorkaroundStruct.init);
 @property ref T lvalueOf(T)(inout __InoutWorkaroundStruct = __InoutWorkaroundStruct.init);
 
@@ -466,7 +467,7 @@ template staticMap(alias F, T...)
 }
 
 // std.exception.assertCTFEable
-version (unittest) package(core)
+version (CoreUnittest) package(core)
 void assertCTFEable(alias dg)()
 {
     static assert({ cast(void) dg(); return true; }());

src/core/lifetime.d

@@ -752,7 +752,7 @@ T* emplace(T, Args...)(void[] chunk, auto ref Args args)
     }
 }
 
-version (unittest)
+version (CoreUnittest)
 {
     //Ambiguity
     private struct __std_conv_S

src/core/sys/windows/basetsd.d

@@ -58,7 +58,7 @@ package mixin template AlignedStr(int alignVal, string name, string memberlist,
     mixin( _alignSpec ~ " struct " ~ name ~" { " ~ _alignSpec ~":"~ memberlist~" }" );
 }
 
-version (unittest) {
+version (CoreUnittest) {
     private mixin AlignedStr!(16, "_Test_Aligned_Str", q{char a; char b;});
     private mixin AlignedStr!(0, "_Test_NoAligned_Str", q{char a; char b;});
 }

src/core/thread/fiber.d

@@ -1594,7 +1594,7 @@ unittest {
     assert( composed.state == Fiber.State.TERM );
 }
 
-version (unittest)
+version (CoreUnittest)
 {
     class TestFiber : Fiber
     {