[PATCH] D157497: feat: Migrate isArch16Bit
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 14 12:34:09 PDT 2023
MaskRay added a comment.
In D157497#4584621 <https://reviews.llvm.org/D157497#4584621>, @Pivnoy wrote:
> This discussion was the main motivation for this change.
> https://discourse.llvm.org/t/rfc-refactor-triple-related-classes/70410/11
> As a result, Triple should become a data class, and its dependencies such as Architecture, Operating System etc. represent in the form of interfaces that can be implemented for the necessary instances.
FWIW I still don't see advantages by switching to the new `llvm::TripleUtils::isArch32Bit` style. How is it better than the current way?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157497/new/
https://reviews.llvm.org/D157497
More information about the cfe-commits
mailing list