[PATCH] D11080: WebAssembly: add placeholder intrinsics header

[Dan Gohman]

sunfish accepted this revision.
This revision is now accepted and ready to land.

================
Comment at: include/llvm/IR/IntrinsicsWebAssembly.td:10
@@ +9,3 @@
+//
+// This file defines all of the WebAssembly-specific intrinsics.
+//
----------------
Triple-slash comments, \file, and \brief? I know no other targets do that, but we could start a new trend :-).

================
Comment at: include/llvm/IR/IntrinsicsWebAssembly.td:14
@@ +13,3 @@
+
+let TargetPrefix = "wasm" in {
+}
----------------
Other targets have a comment like this:

// All intrinsics start with "llvm.wasm.".

which is nice.


http://reviews.llvm.org/D11080