[PATCH] D112940: Add a tool for diffing instruction count + stack size remarks

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 16 00:28:37 PST 2021


jhenderson added a comment.

In D112940#3132444 <https://reviews.llvm.org/D112940#3132444>, @paquette wrote:

> Address review comment + change name to avoid terminal completion issues with llvm-size
>
> (I wanted to keep the "size" part in the name because I want to focus this tool around instruction count)

Thanks. Name change seems reasonable to me.

I noticed the pre-merge checks are failing. It seems to not like the new executable name, or there's something else missing (do you need to add this tool to the list of subsitutions lit performs?).


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

https://reviews.llvm.org/D112940



More information about the llvm-commits mailing list