Page MenuHomePhabricator

Make llvm.objectsize more conservative with null in non-zero address spaces

Authored by george.burgess.iv on Jul 2 2018, 5:59 PM.



NULL in non-zero address spaces is dereferenceable/etc, so we probably shouldn't be assuming that its objectsize is zero.

arsenm: pulling you in to confirm that this change isn't breaking code you care about somehow (re: the comment from you on ). If this does break something, I'm happy to try and find some middle-ground. I'm sort of assuming the intent of that comment was more along the lines of "if this is a gcc-compat hack, we should limit it to addrspace(0), which is presumably the only place where we care," though.

Diff Detail


Event Timeline

nlopes accepted this revision.Jul 3 2018, 3:19 PM
nlopes added a subscriber: nlopes.


This revision is now accepted and ready to land.Jul 3 2018, 3:19 PM


(I'll go ahead and commit assuming my interpretation of arsenm's message was correct. Happy to revert or land follow-ups if not)

This revision was automatically updated to reflect the committed changes.