[PATCH] D100995: [WebAssembly] Put utility functions in Utils directory (NFC)

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 22 19:31:58 PDT 2021


thakis added a comment.

Also, similar to aarch64, this is a bit awkward from a dependency perspective: The files in Utils include headers in Target/WebAssembly which include headers in Target/WebAssembly/MCTargetDesc, which include tablegen-generated headers. So Utils depends on generated headers, and kind of depends on MCTargetDesc, but MCTargetDesc also depends on Utils. It happens to work out, but it's a bit awkward.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100995



More information about the llvm-commits mailing list