[all-commits] [llvm/llvm-project] a51ad8: [OpenMP][FIX] Properly check assume only uses
Johannes Doerfert via All-commits
all-commits at lists.llvm.org
Tue Jan 31 06:14:19 PST 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: a51ad873bfc9f3ccf7971efce4c5f0a543ee3d5a
https://github.com/llvm/llvm-project/commit/a51ad873bfc9f3ccf7971efce4c5f0a543ee3d5a
Author: Johannes Doerfert <johannes at jdoerfert.de>
Date: 2023-01-31 (Tue, 31 Jan 2023)
Changed paths:
M llvm/lib/Transforms/IPO/AttributorAttributes.cpp
M llvm/test/Transforms/Attributor/value-simplify-assume.ll
M llvm/test/Transforms/Attributor/value-simplify-pointer-info.ll
Log Message:
-----------
[OpenMP][FIX] Properly check assume only uses
We improved our simplification and this exposed a bug in the store
elimination. A load that had dead uses and assume uses was thought to be
used by assumes only. Consequently we also deleted the "dead use users".
This was a problem because a dead use just means we will not use the
load there. The user might still be needed.
Exposed by OvO, reported by @ye-luo.
More information about the All-commits
mailing list