[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