[PATCH] D129868: [SystemZInstPrinter] Introduce markup tags emission

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 20 00:30:01 PDT 2022


uweigand added a comment.

Sorry for the delay, I was traveling last week.   Thanks for the use case, it's good to have some validation that the output in this form is actually usable.

As to the implementation, I still believe it would be better to eliminate the `static` routines as outlined in my last comment.   Do you think you could give that a try?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129868/new/

https://reviews.llvm.org/D129868



More information about the llvm-commits mailing list