[llvm-commits] [llvm] r61239 - in /llvm/trunk: docs/LangRef.html include/llvm/Attributes.h lib/Analysis/BasicAliasAnalysis.cpp test/Analysis/BasicAA/nocapture.ll

[Eric Christopher]

>>
>> fwiw the obvious cast to unsigned works, but isn't preferable.
>>
> The alignment is currently stored as 5 bits, which can only produce  
> a 32-bit value. Anything wrong with the blunt cast in this case?
>

Not really I guess, no. Of course could just use 1 << ...?

> Feel free to commit that change if it unbreaks your stuff, I won't  
> get the chance to until end of day tomorrow.
>> What happened to the changes in the patch you were testing?
>>
> Sorry, but I'm not sure what you mean here?

Some patch that went between you and Bill had a completely different  
one.

-eric