[llvm-bugs] [Bug 30216] GVNHoist hoists a store above a load of the same memory location

2016-10-11 Thread via llvm-bugs 
https://llvm.org/bugs/show_bug.cgi?id=30216

Sebastian Pop  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED

--- Comment #14 from Sebastian Pop  ---
Fixed in r283965.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

[llvm-bugs] [Bug 30216] GVNHoist hoists a store above a load of the same memory location

2016-10-10 Thread via llvm-bugs 
https://llvm.org/bugs/show_bug.cgi?id=30216

Sebastian Pop  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---

--- Comment #13 from Sebastian Pop  ---
Reopening, as the fix was reverted in:

commit 579bab4252d27fec369fca49aab7fda023775307
Author: Hans Wennborg 
Date:   Thu Sep 22 21:20:53 2016 +

Revert r282168 "GVN-hoist: fix store past load dependence analysis
(PR30216)"

and also the dependent r282175 "GVN-hoist: do not dereference null
pointers"

It's causing compiler crashes building Harfbuzz (PR30499).

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@282199
91177308-0d34-0410-b5e6-96231b3b80d8

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

[llvm-bugs] [Bug 30216] GVNHoist hoists a store above a load of the same memory location

2016-09-22 Thread via llvm-bugs 
https://llvm.org/bugs/show_bug.cgi?id=30216

Sebastian Pop  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #12 from Sebastian Pop  ---
Fixed in https://reviews.llvm.org/rL282168

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs