[PATCH] D42193: [WebAssembly] Remove DEBUG_FUNCTION_NAME after llvm change

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 17 14:45:15 PST 2018


sbc100 added a comment.

As a rule I always prefer smaller focused changes, but at the same time it can be overwhelming to get so many changes to review at the same time.

I would say that especially for things like typos its nice to separate them since reviewing them is so trivial, and it leaves the real meaty changes with less noise in them.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D42193





More information about the llvm-commits mailing list