diff options
| author | Tim Northover <tnorthover@apple.com> | 2015-04-10 22:58:48 +0000 |
|---|---|---|
| committer | Tim Northover <tnorthover@apple.com> | 2015-04-10 22:58:48 +0000 |
| commit | 1290047a7d70b304114127dbcabe5f79dbca325e (patch) | |
| tree | 233138ba8363bfccda78ab948ec21cd6297c0740 /llvm/unittests/Support/CommandLineTest.cpp | |
| parent | 8106a264d37d2f04423a69250be5cbda0e21ec9c (diff) | |
| download | llvm-1290047a7d70b304114127dbcabe5f79dbca325e.zip llvm-1290047a7d70b304114127dbcabe5f79dbca325e.tar.gz llvm-1290047a7d70b304114127dbcabe5f79dbca325e.tar.bz2 | |
Generic: Make isMask_N and isShiftedMask_N consistent over 0
Previously, isMask_N returned false for 0 but isShiftedMask_N returned true.
Almost all uses are for pattern matching bitfield operations in the backends,
and expect false (this was discovered because of AArch64's copy of this logic).
Unfortunately, I couldn't put together a small non-fragile test for this. The
nature of the bitfield operations means that this edge case is only really
triggered for nodes like "(and N, 0)", which the DAG combiner is usually very
good at folding away before they get to this stage.
rdar://20501377
llvm-svn: 234659
Diffstat (limited to 'llvm/unittests/Support/CommandLineTest.cpp')
0 files changed, 0 insertions, 0 deletions