Thanks for the improvements.

Updating D158180: [mlir][emitc] Add comparison operation

Updating D158180: [mlir][emitc] Add comparison operation

Updating D158180: [mlir][emitc] Add comparison operation

Updating D158180: [mlir][emitc] Add comparison operation

Can you add an error message when the module contains multiple func ops, paired with a test in test/Target/Cpp/invalid.mlir. Currently the translation succeeds but generates code that contains repeated class definitions with the same name.

To me this looks like a lot of additional code that targets a very specific use case. Additionally this hardcodes the assumption that the Tensor class exposes a get method into the emitter.

Thanks for the cleanup @QuqqU. This LGTM.

Sorry for the delay, I got distracted by other work :)

Add tests for the CPP target.

I will add tests for the emitter now that it has landed, as well as a verifier for the opaque type to check whether it's a valid C identifier.

Updating D109143: [mlir] Fix printing of EmitC attrs/types with escape characters

Address review comments.