diff options
| author | Craig Topper <craig.topper@gmail.com> | 2017-03-31 06:30:25 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@gmail.com> | 2017-03-31 06:30:25 +0000 |
| commit | a4f660b669d7f0bb929a5fffc69a3973a29694c0 (patch) | |
| tree | be9aa65f6b39ab87375aadcdb553285a523ca2dc /llvm/lib/CodeGen/MachinePipeliner.cpp | |
| parent | 79235bd4d8c9745b823e8ab4383acc5b81d230bf (diff) | |
| download | llvm-a4f660b669d7f0bb929a5fffc69a3973a29694c0.zip llvm-a4f660b669d7f0bb929a5fffc69a3973a29694c0.tar.gz llvm-a4f660b669d7f0bb929a5fffc69a3973a29694c0.tar.bz2 | |
[APInt] Add unittests that demonstrate how very broken APIntOps::isShiftedMask is.
Did you know that 0 is a shifted mask? But 0x0000ff00 and 0x000000ff aren't? At least we get 0xff000000 right.
I only see one usage of this function in the code base today and its in InstCombine. I think its protected against 0 being misreported as a mask. I guess we just don't have tests for the missed cases.
llvm-svn: 299187
Diffstat (limited to 'llvm/lib/CodeGen/MachinePipeliner.cpp')
0 files changed, 0 insertions, 0 deletions