Re: [PATCH] D71466: [ARM][MVE][Intrinsics] remove extraneous intrinsics.

2019-12-13 Thread Mark Murray via cfe-commits
Hi I've committed a fix. M On 13/12/2019, 17:38, "Nico Weber via Phabricator" wrote: thakis added a comment. This breaks tests: http://45.33.8.238/mac/4132/step_7.txt Please take a look and revert if it takes a while to fix. Repository: rG LLVM

[PATCH] D71466: [ARM][MVE][Intrinsics] remove extraneous intrinsics.

2019-12-13 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. This breaks tests: http://45.33.8.238/mac/4132/step_7.txt Please take a look and revert if it takes a while to fix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71466/new/ https://reviews.llvm.org/D71466

Re: [PATCH] D71466: [ARM][MVE][Intrinsics] remove extraneous intrinsics.

2019-12-13 Thread Mark Murray via cfe-commits
Hi Sorry about the breakage; a commit crossed my build/check/push run, and I was testing a fix when your revert came. M On 13/12/2019, 16:24, "Dmitri Gribenko via Phabricator" wrote: gribozavr2 added a comment. I reverted this change in 34536db7bbe0b8c5f8ffa70df307312b451aca2e

[PATCH] D71466: [ARM][MVE][Intrinsics] remove extraneous intrinsics.

2019-12-13 Thread Dmitri Gribenko via Phabricator via cfe-commits
gribozavr2 added a comment. I reverted this change in 34536db7bbe0b8c5f8ffa70df307312b451aca2e . This change didn't compile: http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/20462. Please always run `ninja

[PATCH] D71466: [ARM][MVE][Intrinsics] remove extraneous intrinsics.

2019-12-13 Thread Mark Murray via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG0eb099273918: [ARM][MVE][Intrinsics] remove extraneous intrinsics. (authored by MarkMurrayARM). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71466/new/

[PATCH] D71466: [ARM][MVE][Intrinsics] remove extraneous intrinsics.

2019-12-13 Thread Mark Murray via Phabricator via cfe-commits
MarkMurrayARM created this revision. MarkMurrayARM added a reviewer: simon_tatham. Herald added subscribers: cfe-commits, dmgreen, kristof.beyls. Herald added a project: clang. I overstepped my reach and generated too many intrinsics; these never made it into the tests. Remove these extras. Some