aboutsummaryrefslogtreecommitdiff
path: root/llvm/tools/llvm-readobj/llvm-readobj.cpp
diff options
context:
space:
mode:
authorserge-sans-paille <sguelton@mozilla.com>2022-12-30 08:32:59 +0100
committerserge-sans-paille <sguelton@mozilla.com>2023-01-12 12:08:06 +0100
commit07bb29d8ffc3b82d5a7bb1217d93e8fa86e6969a (patch)
tree1d8545af548c84b9ff8963f659beb57e854143fd /llvm/tools/llvm-readobj/llvm-readobj.cpp
parentbbe463d6ba268a2bfc45d539314b70cfd72d2360 (diff)
downloadllvm-07bb29d8ffc3b82d5a7bb1217d93e8fa86e6969a.zip
llvm-07bb29d8ffc3b82d5a7bb1217d93e8fa86e6969a.tar.gz
llvm-07bb29d8ffc3b82d5a7bb1217d93e8fa86e6969a.tar.bz2
[OptTable] Precompute OptTable prefixes union table through tablegen
This avoid rediscovering this table when reading each options, providing a sensible 2% speedup when processing and empty file, and a measurable speedup on typical workloads, see: This is optional, the legacy, on-the-fly, approach can still be used through the GenericOptTable class, while the new one is used through PrecomputedOptTable. https://llvm-compile-time-tracker.com/compare.php?from=4da6cb3202817ee2897d6b690e4af950459caea4&to=19a492b704e8f5c1dea120b9c0d3859bd78796be&stat=instructions:u Differential Revision: https://reviews.llvm.org/D140800
Diffstat (limited to 'llvm/tools/llvm-readobj/llvm-readobj.cpp')
-rw-r--r--llvm/tools/llvm-readobj/llvm-readobj.cpp6
1 files changed, 4 insertions, 2 deletions
diff --git a/llvm/tools/llvm-readobj/llvm-readobj.cpp b/llvm/tools/llvm-readobj/llvm-readobj.cpp
index 89f6a89..a11de35 100644
--- a/llvm/tools/llvm-readobj/llvm-readobj.cpp
+++ b/llvm/tools/llvm-readobj/llvm-readobj.cpp
@@ -80,9 +80,11 @@ static constexpr opt::OptTable::Info InfoTable[] = {
#undef OPTION
};
-class ReadobjOptTable : public opt::OptTable {
+class ReadobjOptTable : public opt::GenericOptTable {
public:
- ReadobjOptTable() : OptTable(InfoTable) { setGroupedShortOptions(true); }
+ ReadobjOptTable() : opt::GenericOptTable(InfoTable) {
+ setGroupedShortOptions(true);
+ }
};
enum OutputFormatTy { bsd, sysv, posix, darwin, just_symbols };