[PATCH] D17337: __builtin_object_size problem with pointer as argument

George Burgess IV via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 17 15:46:37 PST 2016


george.burgess.iv added a comment.

Thanks for the patch!

> That said, why not try to evaluate the objectsize call when we're about to remove it?


Agreed. If the `@llvm.objectsize` call points directly to the `alloca` we're trying to kill, then I think it should be trivial to evaluate on the spot. This way, we can go ahead and eliminate the `alloca` instead of hoping that InstCombine runs again to get rid of it for us. :)


Repository:
  rL LLVM

http://reviews.llvm.org/D17337





More information about the llvm-commits mailing list