AArch64, LLVM 11 ERROR: Cannot select some types

[data-man]

Discovered in #6928.

Extracted tests from test/stage1/behavior/vector.zig:

doTheTestReduce(.Min, [4]i64{ 1234567, -386, 0, 3 }, @as(i64, -386));
doTheTestReduce(.Min, [4]u64{ 99, 9999, 9, 99999 }, @as(u64, 9));

doTheTestReduce(.Max, [4]i64{ 1234567, -386, 0, 3 }, @as(i64, 1234567));
doTheTestReduce(.Max, [4]u64{ 99, 9999, 9, 99999 }, @as(u64, 99999));

doTheTestReduce(.Min, [4]f16{ -1.9, 5.1, f16_nan, 100.0 }, f16_nan);
doTheTestReduce(.Min, [4]f32{ -1.9, 5.1, f32_nan, 100.0 }, f32_nan);
doTheTestReduce(.Min, [4]f64{ -1.9, 5.1, f64_nan, 100.0 }, f64_nan);

doTheTestReduce(.Max, [4]f16{ -1.9, 5.1, f16_nan, 100.0 }, f16_nan);
doTheTestReduce(.Max, [4]f32{ -1.9, 5.1, f32_nan, 100.0 }, f32_nan);
doTheTestReduce(.Max, [4]f64{ -1.9, 5.1, f64_nan, 100.0 }, f64_nan);

How to test:zig test -target aarch64-native vector.zig

@LemonBoy's #6928 (comment):

Already fixed in LLVM master, no idea what commit fixed it though.

[kubkon]

@data-man have you submitted a bug report upstream btw? If not, would you mind doing it and pasting a link here? I think it's quite critical to do these ASAP especially if they pertain aarch64 - due to Apple making a move to Apple Silicon, we can expect a lot more aarch64 development happening over at LLVM in the near future, so it's good to put that on their radar.

[data-man]

@kubkon

have you submitted a bug report upstream btw?

No.

If not, would you mind doing it and pasting a link here?

Sorry, I'm not registered on the LLVM's bugzilla and I don't want.

it's good to put that on their radar.

But:

Already fixed in LLVM master, no idea what commit fixed it though.

?

[andrewrk]

That's great it's already fixed, so we will leave this issue to track the disabled test case, and then when LLVM 12 comes out we can re-enable it and close this issue.

[andrewrk]

Note this comment: https://github.com/ziglang/zig/pull/6928/files#r526040373

This is a different problem than the one above.
LLVM's handling of nans across different arches improved a lot after the 12 release.

Some of the disabled code marked with 7138 might be a separate issue.