[llvm-commits] [LLVM, SwitchInst, case ranges] Auxiliary patch #1

Duncan Sands baldrick at free.fr
Thu Jan 5 03:56:44 PST 2012


Hi Stepan,

> Of course. Two patches. The first one extends APInt. The second one extends
> SmallSet.

what's the point of adding an "initialized" field to APInt?  If you need such
a concept you can always introduce your own type wrapping APInt which consists
of a pair: an APInt and an "initialized" field.  Also, why is that change
bundled in with the unrelated ULess struct?  And what's that struct for?

Ciao, Duncan.

>
> -Stepan.
>
> Evan Cheng wrote:
>> Where is the patch? Can you attach it again?
>>
>> Thanks,
>>
>> Evan
>>
>> On Jan 3, 2012, at 10:51 AM, Stepan Dyatkovskiy wrote:
>>
>>> Ping.
>>> --
>>> Truly yours,
>>> Stepan Dyatkovskiy
>>>
>>> 02.01.2012, 13:59, "Stepan Dyatkovskiy"<STPWORLD at narod.ru>:
>>>> Ping again and again :-)
>>>> -Stepan.
>>>>
>>>> 29.12.2011, 21:59, "Stepan Dyatkovskiy"<stpworld at narod.ru>:
>>>>
>>>>> Ping.
>>>>> -Stepan.
>>>>>
>>>>> Stepan Dyatkovskiy wrote:
>>>>>> ping.
>>>>>> Stepan Dyatkovskiy wrote:
>>>>>>> Ping.
>>>>>>>
>>>>>>> Stepan Dyatkovskiy wrote:
>>>>>>>> Ping.
>>>>>>>>
>>>>>>>> -Stepan.
>>> _______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list