yronglin wrote: > LGTM but you should check JSONNodeDumper.cpp as well to see if it needs similar changes. Wow! I've missed that before, I have now added support for JSONNodeDumper.cpp. Many thanks for your review! https://github.com/llvm/llvm-project/pull/88269