Cosmetic fixes.

Address review comments.

Addressed review comments.

See https://github.com/llvm/llvm-project/issues/57698

The operation should have no uses.

Try to adjust wording to incorporate both Uday's and Medhi's suggestions.

Use Mehdi's phrasing.

It removes the node from the linked list and destroy it as far as I can tell?

For reference, the reason I don't like "delete" in this context is that Operations are stored in linked lists. But linked list nomenclature uses "delete" to mean something that's different from deallocation.

Updating phrasing according to Uday's suggestion.

Make opL test case slightly more realistic.

Fold namespaces into prototype.

Adjust commit description.

Thanks for the quick review!

Jeff, could you land this one for me, as well?

Jeff, could you land this one for me? I don't have commit access.

Move escapeString to the "utility structs and functions" section.

Redoing this change, since (at least) MLIR uses \HH for hex escapes.

Adjust test case.

Move static helper function to top level.

Use 'continue' instead of 'if'.

Thanks for fixing this!

Thanks for the fix!

Yes please. Thank you!

Wrapped the failed() directly around the recursive call.

Thanks for the quick review! I don't have write access. Any chance you could submit this? (Same question for the other commits you reviewed?)

Done.

Thanks for the review, River! Fixed the nit.

Good point. Adjusted the comments instead.

Adjust commit message.

Undo, adjust comments.

Ran clang-format.

Thanks for the quick review, River! I don't have write access. Would you be able to commit this change?