[PATCH] D49102: [AccelTable] Provide DWARF5AccelTableStaticData for dsymutil.

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 16 03:56:01 PDT 2018


JDevlieghere added a comment.

In https://reviews.llvm.org/D49102#1163267, @labath wrote:

> This looks good to me. There are some different tradeoffs we could make (speed/templates vs. code size/virtualization), but I don't have an opinion on that, so if noone else does either, I guess this is fine.


It seemed reasonable to pick the same trade-offs as we did for the Apple-style accel tables. The motivation then was that we didn't want to pay for a virtual call for every entry in the list.


https://reviews.llvm.org/D49102





More information about the llvm-commits mailing list