That changes to the param in si_llvm_pack_two_int16() and si_llvm_pack_two_int32_as_int16() should probably have been split out. Anyway with or without that change series is: Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>