[PATCH] Add a dereferencable attribute
Philip Reames
listmail at philipreames.com
Fri Jul 11 14:27:10 PDT 2014
I'm happy with the new encodings. Some of my previous comments still stand (naming, comments, etc..), but the overall structure looks good.
I am uncomfortable with the coupling of dereferenceable and nonnull. I don't have an actual counter example, but could we separate that from the current patch and revisit it separately?
Thinking out loud, I wonder about a truly perverse runtime system that wanted to use loads from null+offset to represent calls to runtime functions. The signal handler could advance the PC and parse the load instruction to "return" a value in the right register. This would be a "dereferenceable" load (very indirectly!), but not non-null. I don't think you could represent this in LLVM for other reasons, but it's the best I could do on the spot. :)
http://reviews.llvm.org/D4449
More information about the llvm-commits
mailing list