[PATCH] D40546: StructurizeCFG: Test for branch divergence correctly

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 25 07:27:57 PST 2018


nhaehnle added a comment.

Yes, it's this change. It's subtle, DivergenceAnalysis has always ended up with dangling pointers, but prior to this change it happened to not matter because we never queried newly created values. This change ended up querying newly created values, and the results differed depending on whether the new allocations happened to reuse the dangling pointers. I uploaded a new version of this change as https://reviews.llvm.org/D43743.


Repository:
  rL LLVM

https://reviews.llvm.org/D40546





More information about the llvm-commits mailing list