[PATCH] Add alignment value to allowsUnalignedMemoryAccess
hfinkel at anl.gov
Sun Jul 27 10:21:40 PDT 2014
----- Original Message -----
> From: "Matt Arsenault" <arsenm2 at gmail.com>
> To: reviews+D4684+public+8ab687474a54a6e1 at reviews.llvm.org
> Cc: llvm-commits at cs.uiuc.edu
> Sent: Sunday, July 27, 2014 12:08:49 PM
> Subject: Re: [PATCH] Add alignment value to allowsUnalignedMemoryAccess
> On Jul 26, 2014, at 5:51 PM, hfinkel at anl.gov wrote:
> > I like the idea of adding the alignment as a function parameter,
> > but I don't understand why you want to rename the function:
> > Unaligned is generally the terminology I use for loads not
> > naturally aligned.
> To me, unaligned means no alignment / align = 1. Something less
> aligned than it wants to be is still aligned, just less aligned.
Okay, fair enough. LGTM.
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory
More information about the llvm-commits