[llvm-commits] [llvm] r58505 - /llvm/trunk/lib/VMCore/Type.cpp

[Chris Lattner]

On Oct 31, 2008, at 3:26 AM, Sanjiv Gupta wrote:

> Author: sgupta
> Date: Fri Oct 31 05:26:20 2008
> New Revision: 58505
>
> URL: http://llvm.org/viewvc/llvm-project?rev=58505&view=rev
> Log:
> For some targets pointer and int are 16-bits. Allow 16-bits as a  
> valid index
> in such cases.

Sanjiv,

Are you planning to make changes to the bc reader/writer, docs, and  
other parts of the compiler to support this change?  I'd strongly  
prefer to you produce a more holistic patch that makes this change  
instead of a couple individual places.

I think this is the right way to go, but I don't want to get stuck  
with half of the change in the tree but it not working 100%.

-Chris