Change ASM Op printer to print the operation name in the framework instead of leaving it up to each individual operation

[mehdi_amini]

Summary

This aligns the printer with the parser contract: the operation isn't part of the user-controllable part of the syntax.

Depends On D108803

[rriddle]

mlir/lib/IR/Operation.cpp

650

Why printEscapedString? AFAIK we aren't going to be able to parse any escaped characters back in.

[mehdi_amini]

Fix for printOneResultOp

[lattner]

Thank you so much for doing this!

[mehdi_amini]

Fix parseOneResultSameOperandTypeOp to support parsing the generic form emitted by printOneResultOp

One consequence of the current patch is that using printGenericOp from a custom printer
requires supporting parsing back the generic form from the custom parser, since the op name
is already printed without quotes.

[mehdi_amini]

Rebase

[mehdi_amini]

Fix flang

[bondhugula]

mlir/lib/IR/AsmPrinter.cpp

385

Is this method named correctly? It's not printing the entire op.

2515

Nit: Print the rest of the op now.

2532

Nit: single quotes

[mehdi_amini]

mlir/lib/IR/AsmPrinter.cpp

385

With the default value printGenericOp(op) is still printing the entire op, I don't feel that having the ability to write printGenericOp(op, /*printOpName=*/false) really warrants a renaming: how would you rename it?

[mehdi_amini]

Address Uday's comments