Fix dead visitor guards in LLVMMemoryCopyFillLowering

src/passes/LLVMMemoryCopyFillLowering.cpp

@@ -246,11 +246,11 @@ struct LLVMMemoryCopyFillLowering
     module->getFunction(memFillFuncName)->body = body;
   }
 
-  void VisitTableCopy(TableCopy* curr) {
+  void visitTableCopy(TableCopy* curr) {
     Fatal() << "table.copy instruction found. Memory copy lowering is not "
                "designed to work on modules with bulk table operations";
   }
-  void VisitTableFill(TableCopy* curr) {
+  void visitTableFill(TableFill* curr) {
     Fatal() << "table.fill instruction found. Memory copy lowering is not "
                "designed to work on modules with bulk table operations";
   }

test/lit/passes/memory-copy-fill-lowering-table-error.wast

@@ -0,0 +1,14 @@
+;; The llvm-memory-copy-fill-lowering pass should Fatal() when a module
+;; contains table.copy, since it does not support bulk table operations.
+
+;; RUN: not wasm-opt --enable-bulk-memory %s --llvm-memory-copy-fill-lowering 2>&1 | filecheck %s
+;; CHECK: table.copy instruction found
+
+(module
+ (memory 0)
+ (table $t 1 funcref)
+ (func $test
+  (memory.copy (i32.const 0) (i32.const 0) (i32.const 0))
+  (table.copy $t $t (i32.const 0) (i32.const 0) (i32.const 0))
+ )
+)

test/lit/passes/memory-copy-fill-lowering-table-fill-error.wast

@@ -0,0 +1,14 @@
+;; The llvm-memory-copy-fill-lowering pass should Fatal() when a module
+;; contains table.fill, since it does not support bulk table operations.
+
+;; RUN: not wasm-opt --enable-bulk-memory --enable-reference-types %s --llvm-memory-copy-fill-lowering 2>&1 | filecheck %s
+;; CHECK: table.fill instruction found
+
+(module
+ (memory 0)
+ (table $t 1 funcref)
+ (func $test
+  (memory.copy (i32.const 0) (i32.const 0) (i32.const 0))
+  (table.fill $t (i32.const 0) (ref.null func) (i32.const 0))
+ )
+)