sameerds added a comment. In D153953#4455794 <https://reviews.llvm.org/D153953#4455794>, @yaxunl wrote:
> Marking mbcnt as convergent, together with https://reviews.llvm.org/D144756 > prevent mbcnt to be merged, which fixed the reported issue. > > Do you have an alternative fix for the issue? I completely disagree with this line of thought. The change to mbcnt is fundamentally incorrect and not related to the issue. There is no ground to ask for changes to this revision. Why was the change to mbcnt committed if was not an actual fix for anything? Also, please note that the issue itself is invalid. I have put a comment in github explaining the same. The original program itself is invalid. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153953/new/ https://reviews.llvm.org/D153953 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
