[PATCH] D67739: [WebAssembly] Let users know that wasm64 does not exist

Dan Gohman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 18 16:22:10 PDT 2019


sunfish added a comment.

Would it be better to do this check in LLVM, in the backend, with a `report_fatal_error`?

Clang for its part already supports what we expect wasm64 will need already, and we have a bunch of tests for it, so it'd be nice to keep that code around and tested. And, doing the check in the LLVM backend also means we'll catch users trying to use it from languages other than C/C++.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67739





More information about the cfe-commits mailing list