[PATCH] D43317: [WebAssebmly] Remove use of Optional to fix assertion in gcc

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 14 14:32:29 PST 2018


ruiu added a comment.

If they can't be 0, 0 is a good initial value that represents "no value". If 0 is a valid value, we can use (uint32_t)-1. We do that in other ports, and it is perhaps better for consistency.

I just removed the failing static_assert to unbreak buildbots. Please add it back to this patch.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D43317





More information about the llvm-commits mailing list