[PATCH] D133109: [LLVM][ARM] Remove options for armv2, 2A, 3 and 3M

[Nick Desaulniers via llvm-commits]

Arnd mentions on IRC:

1:38 AM <arnd> ndesaulniers: I just looked at the patch and found that the
list of supported targets still includes armv5 and armve (both without t),
which never existed in hardware and were removed from gcc a while ago
1:38 AM <arnd> might be good to take them out here as well
1:38 AM <arnd> the only v5 targets in gcc now are armv5t and armv5te
1:40 AM <arnd> not sure about armv5tej, which clang also accepts. I'm
pretty sure that neither clang not gcc actually supported 'j' instructions
as they are not publicly documented, but I don't know if gcc previously
accepted that name as an alias
1:41 AM <arnd> OTOH gcc does allow 'armv6j' and a few others that don't
really exist in hardware without additional letters

On Fri, Sep 2, 2022, 3:56 AM Renato Golin via Phabricator <
reviews at reviews.llvm.org> wrote:

> rengolin accepted this revision.
> rengolin added a comment.
>
> Agree. Even 10 years ago we made the concerted effort not to care about
> pre-v4, so I'd be a little surprised if people are actually using modern
> clang to target those platforms.
>
> Projects that rely on it can work in the same way as gcc and pick an older
> version of the toolchains.
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D133109/new/
>
> https://reviews.llvm.org/D133109
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220902/4153976d/attachment.html>