Artyom Skrobov via llvm-dev
2016-Jan-04 14:16 UTC
[llvm-dev] Diff to add ARMv6L to Target parser
>> However, because the DefaultTargetTriple is armv6l-unknown-linux-gnueabihf, >> and llvm didn’t know about v6l, it would fail to match and canonicalize to armv6. >> I added the notion of v6l to llvm to address this. > > ARMv6l was definitely there once. I'm not sure what happened. > > I'm copying the ARM folks that did most of the recent changes in hope they can > shed some light.Going back through SVN history, I cannot find any evidence that ARMv6L ever existed. There used to be ARMv6HL, until r252903 changed it from an architecture variant into an alias for ARMv6K; i.e. ARMv6HL can still be used in a target triple. For a reference, http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/TargetParser.cpp?revision=238651 is the last version of TargetParser before any of our changes, i.e. authored purely by Renato. One can see it has v6HL but not v6L.
Renato Golin via llvm-dev
2016-Jan-04 14:21 UTC
[llvm-dev] Diff to add ARMv6L to Target parser
On 4 January 2016 at 14:16, Artyom Skrobov <Artyom.Skrobov at arm.com> wrote:> Going back through SVN history, I cannot find any evidence that ARMv6L ever existed.Oh, my bad!! I was thinking of ARMv7l... :/ New year's first brain fart. Sorry about that. Nevertheless, I'll leave you guys to review this one, as I lost touch with the parser a while ago. cheers, --renato
Artyom Skrobov via llvm-dev
2016-Jan-04 14:42 UTC
[llvm-dev] Diff to add ARMv6L to Target parser
>> Going back through SVN history, I cannot find any evidence that ARMv6L ever existed. > > Oh, my bad!! I was thinking of ARMv7l... :/ > > Nevertheless, I'll leave you guys to review this one, as I lost touch with the parser a while ago.Ah, I see: ARMv7L is now an alias for ARMv7A. So, if William has to add support for ARMv6L, I'd suggest he adds it as an alias, and not as an architecture variant.>>> However, because the DefaultTargetTriple is armv6l-unknown-linux-gnueabihfWhere is this "armv6l" coming from? Is it a legacy GNU thing? Or is it a newly-coined name?