diff options
author | Jay Foad <jay.foad@amd.com> | 2019-12-16 10:21:54 +0000 |
---|---|---|
committer | Jay Foad <jay.foad@amd.com> | 2019-12-16 10:25:57 +0000 |
commit | 4f17b1784e94609a2139e0520593884e23345f7e (patch) | |
tree | 227f3c7e2ae6e2f45b0c475b925fdd7c6f47571f /llvm/lib/Transforms/Utils/BasicBlockUtils.cpp | |
parent | 3b9715cb219352fb831af144fd68e14e8fd275b4 (diff) | |
download | bcm5719-llvm-4f17b1784e94609a2139e0520593884e23345f7e.tar.gz bcm5719-llvm-4f17b1784e94609a2139e0520593884e23345f7e.zip |
Fix for AMDGPU MUL_I24 known bits calculation
Summary:
At present, the code calculating known bits of AMDGPU MUL_I24 confuses the concepts of "non-negative number" and "positive number".
In some situations, it results in incorrect code. I have a case where the optimizer replaces the result of calculating MUL_I24(-5, 0) with -8.
Reviewers: foad, arsenm
Reviewed By: arsenm
Subscribers: foad, arsenm, kzhuravl, jvesely, wdng, nhaehnle, yaxunl, dstuttard, tpr, t-tye, hiraditya, llvm-commits
Tags: #llvm
Patch by Eugene Kuznetsov.
Differential Revision: https://reviews.llvm.org/D70367
Diffstat (limited to 'llvm/lib/Transforms/Utils/BasicBlockUtils.cpp')
0 files changed, 0 insertions, 0 deletions