| Message ID | 1371819691-27308-1-git-send-email-peter.maydell@linaro.org |
|---|---|
| State | New |
| Headers | show |
On 06/21/2013 06:01 AM, Peter Maydell wrote: > clang 3.8 with -fsanitize=undefined will fail to link code containing an > int128_t * int128_t multiply (http://llvm.org/bugs/show_bug.cgi?id=16404) > so add this to our configure test for whether [u]int128_t are usable. > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Richard Henderson <rth@twiddle.net> r~
On 21 June 2013 14:01, Peter Maydell <peter.maydell@linaro.org> wrote: > clang 3.8 with -fsanitize=undefined will fail to link code containing an > int128_t * int128_t multiply (http://llvm.org/bugs/show_bug.cgi?id=16404) > so add this to our configure test for whether [u]int128_t are usable. I meant "clang 3.3" here -- can this be fixed when the patch is committed to -trivial or do you want a respin? thanks -- PMM
21.06.2013 17:01, Peter Maydell wrote: > clang 3.8 with -fsanitize=undefined will fail to link code containing an > int128_t * int128_t multiply (http://llvm.org/bugs/show_bug.cgi?id=16404) > so add this to our configure test for whether [u]int128_t are usable. I accept it as granted (with the fixup s/3.8/3.3/), I didn't try to verify whenever it's actually true for clang. Applied to the trivial patches queue (with the fixup), thanks! /mjt
diff --git a/configure b/configure index ad32f87..6465e05 100755 --- a/configure +++ b/configure @@ -3329,6 +3329,7 @@ __uint128_t b; int main (void) { a = a + b; b = a * b; + a = a * a; return 0; } EOF
clang 3.8 with -fsanitize=undefined will fail to link code containing an int128_t * int128_t multiply (http://llvm.org/bugs/show_bug.cgi?id=16404) so add this to our configure test for whether [u]int128_t are usable. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> --- configure | 1 + 1 file changed, 1 insertion(+)