[PATCH] D65280: Add a pass to lower is.constant and objectsize intrinsics

Hans Wennborg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 28 07:44:51 PDT 2019


hans added a comment.

> I'm also against putting it into 9.0, which is supposed to have a final release Real Soon Now. This is not a obviously-correct change, and it should bake in trunk for at least a couple weeks before going into the release, to shake out any unexpected problems.
> 
> I think we //should// target this for the 9.0.1 patch though, which is why joerg wants to merge the API change to 9.0 now.

This sounds good to me. From my point of view, ideally the API change would be broken out and landed on trunk as soon as possible, and then I'd merge that change to the 9 branch.


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

https://reviews.llvm.org/D65280





More information about the llvm-commits mailing list