[PATCH] D86349: [Attributor] Fix IntegerRangeState::unionKnown

Shinji Okumura via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 21 06:28:18 PDT 2020


okura created this revision.
Herald added a reviewer: uenoku.
Herald added subscribers: llvm-commits, kuter, uenoku.
Herald added a reviewer: homerdin.
Herald added a project: LLVM.
okura requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added a reviewer: sstefan1.
Herald added a reviewer: baziotis.
Herald added a subscriber: bbn.

Currently, the assumed range is taken union with a new known range in `IntegerRangeState::unionKnown`.
However, that is unnecessary because an assumed range is guaranteed to be subsumed by a known range.


https://reviews.llvm.org/D86349

Files:
  llvm/include/llvm/Transforms/IPO/Attributor.h
  llvm/test/Transforms/Attributor/potential.ll
  llvm/test/Transforms/Attributor/range.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86349.287020.patch
Type: text/x-patch
Size: 17177 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200821/eb197d2d/attachment.bin>


More information about the llvm-commits mailing list