[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