[PATCH] D44313: [WebAssembly] Implement GC for imports

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 9 11:09:02 PST 2018


sbc100 added inline comments.


================
Comment at: wasm/Symbols.cpp:89
+  }
+  Live = true;
 }
----------------
sbc100 wrote:
> Maybe assert `isUndefined` here?
Alternatively we might want to set this unconditionally at the top of the function?

In which case isLive becomes just `return Live`?



Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D44313





More information about the llvm-commits mailing list