[PATCH] D48015: ARM, AArch64: Use generic tables instead of SearchableTable

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 11 07:52:37 PDT 2018


nhaehnle added a comment.

Sorry for the lack of context.

With the parent revision, SearchableTable is implemented entirely in terms of the new primitives. So the idea is to remove SearchableTable to have only One Way of doing things rather than a proliferation of different approaches to expressing the same thing.

If people feel strongly that using GenericTable + SearchIndex is too verbose relative to SearchableTable, we can leave SearchableTable in. I just thought it'd be cleaner to get rid of it, given that it has become redundant.


Repository:
  rL LLVM

https://reviews.llvm.org/D48015





More information about the llvm-commits mailing list