Change-Id: If7cec95d7d7e6335bbf81f5c589a8e14de9375f9
Details
Diff Detail
- Repository
- rL LLVM
- Build Status
Buildable 19461 Build 19461: arc lint + arc unit
Event Timeline
Its not clear to me why 'GenericTable' is more efficient/better than SearchableTable. Maybe you could give some background in the commit message about this. I implemented the ARMSystemRegister bit sometime ago and found SearchableTable quite useful and concise but perhaps GenericTable/Enum is even better.
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.
Well, yes. I linked the revisions, but Phabricator's support for patch series doesn't seem to be widely used. It's what you see when you click on "Stack" in the overview.