[all-commits] [llvm/llvm-project] e5f602: [ValueTracking] Let propagatesPoison support binop...
Juneyoung Lee via All-commits
all-commits at lists.llvm.org
Tue May 12 10:52:02 PDT 2020
Branch: refs/heads/master
Home: https://github.com/llvm/llvm-project
Commit: e5f602d82ca05d113c725fdfff405dcb7c5470f5
https://github.com/llvm/llvm-project/commit/e5f602d82ca05d113c725fdfff405dcb7c5470f5
Author: Juneyoung Lee <aqjune at gmail.com>
Date: 2020-05-13 (Wed, 13 May 2020)
Changed paths:
M llvm/include/llvm/Analysis/ValueTracking.h
M llvm/lib/Analysis/ValueTracking.cpp
M llvm/lib/Transforms/Instrumentation/PoisonChecking.cpp
M llvm/test/Analysis/ScalarEvolution/nsw.ll
M llvm/unittests/Analysis/ValueTrackingTest.cpp
Log Message:
-----------
[ValueTracking] Let propagatesPoison support binops/unaryops/cast/etc.
Summary:
This patch makes propagatesPoison be more accurate by returning true on
more bin ops/unary ops/casts/etc.
The changed test in ScalarEvolution/nsw.ll was introduced by
https://github.com/llvm/llvm-project/commit/a19edc4d15b0dae0210b90615775edd76f021008 .
IIUC, the goal of the tests is to show that iv.inc's SCEV expression still has
no-overflow flags even if the loop isn't in the wanted form.
It becomes more accurate with this patch, so think this is okay.
Reviewers: spatel, lebedev.ri, jdoerfert, reames, nikic, sanjoy
Reviewed By: spatel, nikic
Subscribers: regehr, nlopes, efriedma, fhahn, javed.absar, llvm-commits, hiraditya
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D78615
More information about the All-commits
mailing list