[PATCH] D44393: [WebAssembly] Add missing implementation for --initial/max-memory args

Nicholas Wilson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 13 06:38:07 PDT 2018


ncw added a comment.

In https://reviews.llvm.org/D44393#1035246, @sbc100 wrote:

> Code looks good.  I have concerns about adding flags that don't have current users.   Can you point to at least one before we add this?


Good question - it's not so much adding a flag, as fixing an existing broken flag, but you're right. I'm not aware of any current users, but I was planning to use it myself in my own projects, to help catching any memory leaks.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D44393





More information about the llvm-commits mailing list