Merge pull request #347 from ragnr/topic/sizeof-unsigned

Correct sizeof for unsigned types. Fixes #328

nscplugin/src/main/scala/scala/scalanative/nscplugin/NirCodeGen.scala

@@ -1414,10 +1414,9 @@ abstract class NirCodeGen
       val Apply(_, Seq(ctp)) = app
 
       val sym = extractClassFromImplicitClassTag(ctp)
-      val ty  = genTypeSym(sym)
 
       code match {
-        case SIZEOF => focus withOp Op.Sizeof(ty)
+        case SIZEOF => focus withOp Op.Sizeof(genTypeSym(sym, boxUnsigned = false))
         case TYPEOF => focus withValue genTypeSymValue(sym)
         case _      => unreachable
       }

unit-tests/src/main/scala/scala/scalanative/native/UnsignedSizeOfSuite.scala

@@ -0,0 +1,21 @@
+package scala.scalanative.native
+
+object UnsignedSizeOfSuite extends tests.Suite {
+
+  test("UByte's size should be 1") {
+    assert(sizeof[UByte] == 1)
+  }
+
+  test("UShort's size should be 2") {
+    assert(sizeof[UShort] == 2)
+  }
+
+  test("UInt's size should be 4") {
+    assert(sizeof[UInt] == 4)
+  }
+
+  test("ULong's size should be 8") {
+    assert(sizeof[ULong] == 8)
+  }
+
+}

unit-tests/src/main/scala/tests/Main.scala

@@ -16,6 +16,7 @@ object Main {
       scala.scalanative.native.CStringSuite,
       scala.scalanative.native.CInteropSuite,
       scala.scalanative.native.InstanceOfSuite,
+      scala.scalanative.native.UnsignedSizeOfSuite,
       scala.ArrayIntCopySuite,
       scala.ArrayDoubleCopySuite,
       scala.ArrayObjectCopySuite,