ymandel marked 3 inline comments as done. ymandel added a comment. In D122231#3414109 <https://reviews.llvm.org/D122231#3414109>, @xazax.hun wrote:
> Wow. This did take some iterations and I feel like I just added to the > confusion at some point :D But the latest iteration looks much simpler and > I'm confident it is right this time. Thanks! Not at all -- I think you raised some really good questions! Ultimately, my move from implication in the flow condition to tying it directly to the value was the wrong turn, and your questions effectively highlighted the issues. :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122231/new/ https://reviews.llvm.org/D122231 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits