xazax.hun added a comment.

This fix looks good for me for this particular problem, but I wonder whether 
the solution is general enough. In case the analysis figures out that a call 
would not return (e.g., the `value` method is called on a provably empty 
optional, and it would throw instead of returning), would this approach still 
work? Would the analysis update the `BlockReachable` bitvector on demand?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146514/new/

https://reviews.llvm.org/D146514

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to