[llvm-commits] PR10663: remove DEFINE_TRANSPARENT_CASTED_OPERAND_ACCESSORS

Gabor Greif ggreif at gmail.com
Fri Aug 19 13:25:19 PDT 2011


Ahhhh, I guess I see:
you did not touch the DECLARE_* macros, but eliminated the redundant
DEFINE_T_CASTED_O_A one. I guess this is fine with me :-)

Cheers,

    Gabor


On 8/19/11, Jay Foad <jay.foad at gmail.com> wrote:
> Hi Gabor,
>
> On 19 August 2011 19:36, Gabor Greif <ggreif at gmail.com> wrote:
>> as I commented in the PR, I prefer not to remove DTCOA, because before
>> they were introduced ConstantArray::getOperand() also returned
>> Constant*, not Value*.
>
> With my patch, ConstantArray::getOperand() still returns Constant*.
>
> Jay.
>



More information about the llvm-commits mailing list