jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land.
In D124724#3486921 <https://reviews.llvm.org/D124724#3486921>, @tianshilei1992 wrote: > always use `UIToFP` because that is the only case that `UpdateVal` is a > `ConstantInt`, where we created the constant integer for `++` or `--` > operation. Makes sense. LG. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124724/new/ https://reviews.llvm.org/D124724 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits