[PATCH] D18738: Add new !unconditionally_dereferenceable load instruction metadata
Philip Reames via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 18 18:03:10 PDT 2016
reames added a comment.
Can you give a bit of context on what the source level rule which implies global dereferenceability is?
The patch is presented looks mostly okay - I haven't gone through it carefully yet - but the motivation bothers me. Having a condition load from something which isn't obviously dereferenceable (i.e. global, etc..) seems to imply we might be missing something. i.e. why was it conditional at the source level to start with?
Repository:
rL LLVM
http://reviews.llvm.org/D18738
More information about the llvm-commits
mailing list