[PATCH] D86360: Add new hidden option -print-changed which only reports changes to IR
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 17 22:25:51 PDT 2020
MaskRay added a comment.
I think my biggest question is about the necessity of the ChangePrinter base class... The paragraph in the summary hints on that it is probably necessary....... but the link is unreadable and I think we need other concrete derived classes to justify the base class design.
> The first of several instantiations is provided that prints the IR in a form similar to that produced by -print-after-all with the above mentioned filtering capabilities. This version, and the others to follow will be introduced at the upcoming developer's conference. See https://hotcrp.llvm.org/usllvm2020/paper/29 for more information.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D86360/new/
https://reviews.llvm.org/D86360
More information about the llvm-commits
mailing list