Using the "LHS" variable in the splitBinaryBitConstantOp() call is not valid since "LHS" and "RHS" may be swapped earlier in the code.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Time | Test | |
---|---|---|
10,140 ms | x64 debian > libFuzzer.libFuzzer::fuzzer-custommutator.test |
Event Timeline
Comment Actions
LGTM with a nit below. Thanks!
llvm/test/CodeGen/AMDGPU/combine-reg-or-const.ll | ||
---|---|---|
1 ↗ | (On Diff #385086) | Can you auto generate this test before submitting? |
llvm/test/CodeGen/AMDGPU/combine-reg-or-const.ll | ||
---|---|---|
1 ↗ | (On Diff #385086) | Alternatively, you can add more checks for more context, thanks. |
clang-format: please reformat the code