[PATCH] D44150: [WebAssembly] Move WasmSignatureDenseMapInfo to header for reuse. NFC
Sam Clegg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 8 08:38:06 PST 2018
sbc100 accepted this revision.
sbc100 added a comment.
This revision is now accepted and ready to land.
Having two copies seems fine for now.
================
Comment at: wasm/WriterUtils.h:24
+// Needed for DenseMapInfo<wasm::WasmSignature>
inline bool operator==(const llvm::wasm::WasmSignature &LHS,
const llvm::wasm::WasmSignature &RHS) {
----------------
Do these need to be moved into the namespace? if so can you drop the extra llvm:: prefixes?
================
Comment at: wasm/WriterUtils.h:65
+ const wasm::WasmSignature &RHS) {
+ return LHS == RHS;
+ }
----------------
Should we just inline this and drop the operators overloads for signatures? Or are they used elsewhere? (In which case the comment neeeds updating I guess).
Repository:
rLLD LLVM Linker
https://reviews.llvm.org/D44150
More information about the llvm-commits
mailing list