[PATCH] D35821: Have ARM call setBooleanContents(ZeroOrOneBooleanContent).
Joel Galenson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 24 16:22:39 PDT 2017
jgalenson created this revision.
Herald added subscribers: kristof.beyls, javed.absar, aemerson.
As pointed out by Eli Friedman, the ARM backend should call setBooleanContents so that it can use known bits to make some optimizations.
In addition, this patch also contains a simple optimization to avoid duplicate ands in one of the test cases.
Unfortunately, this change causes a number of the existing tests to stop being useful. All but the last of the modified tests no longer test what they initially intended to do, mostly due to different undef handling (e.g., cse-call no longer generates any cmps).
I added a new testcase where the first mov is caused by the new code.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 4220 bytes
Desc: not available
More information about the llvm-commits