[llvm-bugs] [Bug 36529] MemorySSA doesn't properly invalidate def-def cache entries

via llvm-bugs llvm-bugs at lists.llvm.org
Mon Feb 26 23:24:45 PST 2018


https://bugs.llvm.org/show_bug.cgi?id=36529

George Burgess <george.burgess.iv at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from George Burgess <george.burgess.iv at gmail.com> ---
Oh, right. VHes are things.

In general, I'm a bit uneasy about the idea that we just hold on to this cache
entry without any way to update/detect it from the cached value, but I can't
think of a concrete example (that isn't a misoptimization) that would cause a
WeakVH approach to break here.

That said, should we find such an example in the future, we can always try to
make that work.

r326175 + r326177 should fix this.

Thank you!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20180227/04e89d5b/attachment.html>


More information about the llvm-bugs mailing list