FailedChanges

Summary

  1. [X86] Regenerate i128 sdiv tests and add i686 coverage. (details)
  2. [X86] Add some basic i128 udiv test coverage (details)
  3. [CVP] Pass context instruction when narrowing div/rem (details)
  4. [LVI] Require context instruction in external API (NFCI) (details)
  5. [LVI] Clarify getValueAt/getValueInBlock doc comments (NFC) (details)
Commit d9499c88d19bd38b17d9d7423c0d476bc77ebcb7 by llvm-dev
[X86] Regenerate i128 sdiv tests and add i686 coverage.

To hopefully help improve the codegen delta in D87976
The file was modifiedllvm/test/CodeGen/X86/i128-sdiv.ll
Commit 2d7a12f4ecfb5e0393ab0cf4ab4f46c198e808a7 by llvm-dev
[X86] Add some basic i128 udiv test coverage
The file was addedllvm/test/CodeGen/X86/i128-udiv.ll
Commit c8abf1c12dc39f3b07b887808da4bb09d5ddbe3e by nikita.ppv
[CVP] Pass context instruction when narrowing div/rem

This fold was the only place not passing the context instruction.
The tests worked around that fact by introducing a basic block split,
which is now no longer necessary.
The file was modifiedllvm/test/Transforms/CorrelatedValuePropagation/sdiv.ll
The file was modifiedllvm/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp
Commit 9b959b59dfaf87ba978480594d2cfcf15fe66218 by nikita.ppv
[LVI] Require context instruction in external API (NFCI)

Require CxtI in getConstant() and getConstantRange() APIs.
Accordingly drop the BB parameter, as it is implied by
CxtI->getParent().

This makes sure we don't forget to pass the context instruction,
and makes the API contract clearer (also clean up the comments to
that effect -- the value holds at the context instruction, not
the end of the block).
The file was modifiedllvm/lib/Analysis/LazyValueInfo.cpp
The file was modifiedllvm/include/llvm/Analysis/LazyValueInfo.h
The file was modifiedllvm/lib/Transforms/Scalar/JumpThreading.cpp
The file was modifiedllvm/lib/Transforms/Utils/LowerSwitch.cpp
The file was modifiedllvm/lib/Transforms/IPO/AttributorAttributes.cpp
The file was modifiedllvm/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp
Commit 709d03f8af4da4204849a70f01798e7cebba2e32 by nikita.ppv
[LVI] Clarify getValueAt/getValueInBlock doc comments (NFC)

The lattice value returned by getValueInBlock() holds at the start
of the block, not at the end. Also make it clearer what the
difference between getValueInBlock() and getValueAt() is.
The file was modifiedllvm/lib/Analysis/LazyValueInfo.cpp