Merge pull request #39 from rogers0/PR/refine_32bit

Refine 32-bit support
viant · Oct 12, 2020 · c12f88e · c12f88e
2 parents a08a685 + 010a97e
commit c12f88e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/data/udf/conversion_test.go b/data/udf/conversion_test.go
@@ -1,7 +1,6 @@
 package udf
 
 import (
-	"github.com/klauspost/cpuid"
 	"github.com/stretchr/testify/assert"
 	"github.com/viant/toolbox/data"
 	"log"
@@ -139,9 +138,6 @@ func Test_AsString(t *testing.T) {
 	aMap.Put("k0", true)
 	expanded := aMap.ExpandAsText(" $AsString(${k0})")
 	assert.EqualValues(t, "true", expanded)
-	if cpuid.CPU.CacheLine < 64 {
-		return
-	}
 	//64 bit int
 	aMap.Put("k1", 2323232323223)
 	expanded = aMap.ExpandAsText(" $AsString(${k1})")

diff --git a/fileset_info_test.go b/fileset_info_test.go
@@ -1,15 +1,14 @@
 package toolbox_test
 
 import (
-	"github.com/klauspost/cpuid"
 	"github.com/stretchr/testify/assert"
 	"github.com/viant/toolbox"
 	"testing"
 )
 
 func TestNewFileSetInfoInfo(t *testing.T) {
-	if cpuid.CPU.CacheLine < 64 {
-		return
+	if (32 << uintptr(^uintptr(0)>>63)) < 64 {
+		t.Skip()
 	}
 	fileSetInfo, err := toolbox.NewFileSetInfo("./test/fileset_info/")
 	if err != nil {