Merge pull request #1527 from Idclip/32bit_ax_fix

Idclip · web-flow · commit 4dc19b1ef7dc · 2022-12-02T16:06:27.000Z
Fixed a build issue with AX on 32 bit platforms
diff --git a/openvdb_ax/openvdb_ax/codegen/Types.h b/openvdb_ax/openvdb_ax/codegen/Types.h
@@ -198,8 +198,8 @@ struct LLVMType<void>
     }
 };
 
-/// @note void* implemented as signed int_t* to match clang IR generation
-template <> struct LLVMType<void*> : public LLVMType<int_t<sizeof(void*)>::type*> {};
+/// @note void* implemented as signed int8_t* to match clang IR generation
+template <> struct LLVMType<void*> : public LLVMType<int8_t*> {};
 template <> struct LLVMType<openvdb::math::half>
 {
     // @note LLVM has a special representation of half types. Don't alias to
diff --git a/pendingchanges/ax32fix.txt b/pendingchanges/ax32fix.txt
@@ -0,0 +1,3 @@
+Build:
+ - Fixed a build issue with AX on 32bit platforms.
+ [Reported by Mathieu Malaterre]

Original file line number	Diff line number	Diff line change
`@@ -198,8 +198,8 @@ struct LLVMType<void>`
`198`	`198`	`}`
`199`	`199`	`};`
`200`	`200`
`201`		`-/// @note void* implemented as signed int_t* to match clang IR generation`
`202`		`-template <> struct LLVMType<void> : public LLVMType<int_t<sizeof(void)>::type*> {};`
	`201`	`+/// @note void* implemented as signed int8_t* to match clang IR generation`
	`202`	`+template <> struct LLVMType<void> : public LLVMType<int8_t> {};`
`203`	`203`	`template <> struct LLVMType<openvdb::math::half>`
`204`	`204`	`{`
`205`	`205`	`// @note LLVM has a special representation of half types. Don't alias to`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+Build:`
	`2`	`+ - Fixed a build issue with AX on 32bit platforms.`
	`3`	`+ [Reported by Mathieu Malaterre]`