[PATCH] D64322: [WebAssembly] Add support for llvm.clear_cache intrinsic

Derek Schuff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 9 11:36:11 PDT 2019


dschuff added a comment.

Another view would be that we accept that this probably doesn't get used in code that would be useful in wasm, but maybe it's in a file that has other useful code, so this would basically be a convenience feature that allows users to have fewer ifdefs in their files (even with the assumption that they aren't going to be calling this code). Someone did, after all, actually run into this problem. Speaking of, maybe we should ask them what their use case is, why they even want to compile this code?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64322





More information about the llvm-commits mailing list