https://github.com/QuantumSegfault commented:

Other than a couple of nit-picks, it seems fine. But again, I'm not familiar 
with Clang.

Though I notice the only place that `WebAssemblyTableType` element type is 
checked is during Sema? No other guard rails against the internals of Clang 
creating invalid tables (with non-reference types). Probably not an issue. Just 
an observation.

https://github.com/llvm/llvm-project/pull/205817
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to