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

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 30 11:56:17 PDT 2017


MatzeB added a comment.

Some thoughts:

- Is this motivated by AMDGPU having a huge number of registers in the mask?
- 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.


https://reviews.llvm.org/D34897





More information about the llvm-commits mailing list