<div dir="auto">Arnd mentions on IRC:<div dir="auto"><br></div><div dir="auto">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</div><div dir="auto">1:38 AM <arnd> might be good to take them out here as well</div><div dir="auto">1:38 AM <arnd> the only v5 targets in gcc now are armv5t and armv5te</div><div dir="auto">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</div><div dir="auto">1:41 AM <arnd> OTOH gcc does allow 'armv6j' and a few others that don't really exist in hardware without additional letters</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 2, 2022, 3:56 AM Renato Golin via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rengolin accepted this revision.<br>
rengolin added a comment.<br>
<br>
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.<br>
<br>
Projects that rely on it can work in the same way as gcc and pick an older version of the toolchains.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D133109/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D133109/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D133109" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D133109</a><br>
<br>
</blockquote></div>