[PATCH] D12853: Introduce !value.align metadata for load instruction

hfinkel@anl.gov via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 22 14:59:30 PDT 2015


hfinkel added a comment.

I assume that you want to call this '!value.align' to differentiate it from the existing 'align' argument to the load. This is syntactically unnecessary, and while I agree that it could seem confusing, is also inconsistent with the other similar metadata. We don't have value.nonnull or value.dereferenceable, and I'm inclined to think that we should just call this !align for consistency.


http://reviews.llvm.org/D12853





More information about the llvm-commits mailing list