[PATCH] Add alignment value to allowsUnalignedMemoryAccess

Hal Finkel 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.

 -Hal

> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory



More information about the llvm-commits mailing list