[PATCH] D59281: [WebAssembly] "atomics" feature requires shared memory

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added inline comments.


================
Comment at: lld/wasm/Writer.cpp:915
+  if (TargetFeatures.count("atomics"))
+    Config->SharedMemory = true;
+
----------------
I think we decided against this part right?    i.e. atomics should require but not imply shared memory?  So this should probably be an error instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59281