32-bit linux compilation error

[n3f4s]

Hi,
I tried to compile ponyc from source.
I have an error:

$ git clone https://github.com/ponylang/ponyc.git
$ make
src/libponyc/codegen/genident.c: In function 'gen_numeric_size_table':
src/libponyc/codegen/genident.c:453:26: error: conversion to 'size_t {aka unsigned int}' from 'long long unsigned int' may alter its value [-Werror=conversion]
       size_t type_size = LLVMABISizeOfType(c->target_data, t->use_type);

I'm on manjaro linux 32bits (a distribution based on arch linux).

$ uname -a
Linux manjaro 4.9.13-1-MANJARO #1 SMP PREEMPT Sun Feb 26 17:52:50 UTC 2017 i686 GNU/Linux

[SeanTAllen]

What version of LLVM are you using?

[n3f4s]

I'm using llvm 3.9.1-2

$pacman -Qs llvm
local/clang 3.9.1-2
    C language family frontend for LLVM
local/llvm 3.9.1-2
    Low Level Virtual Machine
local/llvm-libs 3.9.1-2
    Low Level Virtual Machine (runtime libraries)

[SeanTAllen]

@Praetonus looks like #1726 broke this.

[SeanTAllen]

@n3f4s sorry for that problem, we should have a fix soon. We don't have 32-bit CI so we do best effort to keep it working.

[SeanTAllen]

@jemc @killerswan per travis-ci/travis-ci#5770, could we use a docker image to 32 bit testing even if it's just with a single LLVM version?

If this seems reasonable, I'll open an issue for it.

[killerswan]

@SeanTAllen @jemc That's an interesting possibility! But now I need to go down the rabbit hole of understanding what testing we already do of our existing Dockerfile anyways, too...

[jemc]

@SeanTAllen @killerswan - not 100% sure, but I don't think Dockerfiles can be used cross-arch.

[killerswan]

What that Travis CI thread is talking about (if I follow correctly) is using 32-bit docker images on 64-bit Travis VMs, though. It'd take making another Dockerfile, I assume, so I was then wondering what else we should be thinking of testing...

[killerswan]

@n3f4s sorry we're hijacking your ticket here! 😅

[Praetonus]

It should be fixed in #1762. Sorry for the trouble.

[SeanTAllen]

@n3f4s the fixed has been merged

[SeanTAllen]

i'm not going to kick off a new release for this as the bug was introduced since the last release so merely updating master should be good enough.