[llvm-commits] [patch][pr12251] Add range metadata to llvm. Make clang produce it.

[Rafael Espíndola]

> OK, that's fair.  One good thing about [A, B) is that someone working with
> i32
> who wants to represent [0, 2^32-1] will say to themselves: "oh no, I need to
> output [0, 2^32) but I can't represent that in 32 bits", and then realize
> that
> they were being silly wanting to output this range in the first place :)

That is true, that was my first reaction to using half open ranges :-)
Nick (on IRC) also mentioned that he prefers half open ranges, so this
new pair of patches uses it.

> Ciao, Duncan.

Cheers,
Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm.patch
Type: text/x-patch
Size: 7063 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120323/c1ddd088/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang.patch
Type: text/x-patch
Size: 7350 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120323/c1ddd088/attachment-0001.bin>