[PATCH] D123484: [WebAssembly] Implement ref.is_null MC layer support and codegen

[Heejin Ahn via Phabricator via llvm-commits]

aheejin accepted this revision.
aheejin added a comment.

Thanks! I have one more questions but don't want to hold this back with nitpicky things. (The review turnaround time due to time zone doesn't help either)



================
Comment at: llvm/include/llvm/IR/IntrinsicsWebAssembly.td:34-37
+def int_wasm_ref_is_null_extern : Intrinsic<[llvm_i32_ty], [llvm_externref_ty],
+                                            [IntrNoMem], "llvm.wasm.ref.is_null.extern">;
+def int_wasm_ref_is_null_func : Intrinsic<[llvm_i32_ty], [llvm_funcref_ty],
+                                          [IntrNoMem], "llvm.wasm.ref.is_null.func">;
----------------
Why do we need to specify names explicitly? Aren't these automatically named like these anyway?


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

https://reviews.llvm.org/D123484