[PATCH] D118121: [WebAssembly] Refactor WasmSymbol type setting code

[Sam Clegg via Phabricator via llvm-commits]

sbc100 accepted this revision.
sbc100 added a comment.
This revision is now accepted and ready to land.

Is this change completely non-functional?  If so can you add NFC to the title?



================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyFastISel.cpp:20
+#include "Utils/WebAssemblyTypeUtilities.h"
 #include "Utils/WebAssemblyUtilities.h"
 #include "WebAssembly.h"
----------------
Are both WebAssemblyUtilities.h and WebAssemblyTypeUtilities.h needed here?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118121/new/

https://reviews.llvm.org/D118121