[PATCH] D34897: Change default for -print-whole-regmask to false

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 30 12:01:14 PDT 2017


arsenm added a comment.

In https://reviews.llvm.org/D34897#797095, @MatzeB wrote:

> Some thoughts:
>
> - Is this motivated by AMDGPU having a huge number of registers in the mask?


Yes. It more than fills my fullscreen terminal window on any call. It wouldn't be so bad if it printed just regunits, but this also prints all of the register tuple combinations as well.

> - For X86, AArch64, ARM the output certainly takes up a bunch of space but I find the information important enough that I'd like to see it by default
> - How about a compromise: We stop using an On/Off switch but instead influence the number of registers printed in the regmask instead of the hardcoded 10 at the moment. Then choose a big default value that easily fits X86, AArch64, ARM etc.

The MIR printer comes up with a symbolic name for the regmask. I was wondering why the debug dump doesn't use that


https://reviews.llvm.org/D34897





More information about the llvm-commits mailing list