[PATCH] D26621: [AArch64] Refactor LSE support as feature separate from V8.1a support.

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 25 08:59:21 PST 2016


rengolin added a comment.

In https://reviews.llvm.org/D26621#605013, @t.p.northover wrote:

> OK, I eventually managed to track it down to "large system extensions", I think we should include that somewhere. Probably alongside noting that it's the atomics because the link is decidedly non-obvious even with the expanded name.


I think the current description is better than if it just said "Large System Extension", as it actually tells you what it is about. :)

I don't mind as it is, or an additional comment just about the declaration. Everything else looks fine, too. Leaving to Tim to approve.

cheers,
--renato


https://reviews.llvm.org/D26621





More information about the llvm-commits mailing list