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

George Burgess <george.burgess...@gmail.com> changed:

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

--- Comment #2 from George Burgess <george.burgess...@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.
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to