[LLVMdev] [patch] MicroBlaze Backend

Anton Korobeynikov anton at korobeynikov.info
Sat Jan 30 10:59:02 PST 2010

Hello, Wesley

>> More comments:
>>> +SDValue MBlazeTargetLowering::
>>> +LowerDYNAMIC_STACKALLOC(SDValue Op, SelectionDAG &DAG) {
>> Do you really need this? Expanding dynamic allocas normally ends with
>> stack register adjustment, you don't need anything special here.
> I was unsure as to whether I needed this or not. I have been taking the approach of removing things conservatively as I continue to improve the backend. I will take a look at removing this for the next patch.
Well, the code to trigger this is quite simple:

define void @t() nounwind {
        br label %entry

        %m1 = alloca i32, align 4
        %m2 = alloca [7 x i8], align 16
        call void @s( i32* %m1, [7 x i8]* %m2 )
        ret void

declare void @s(i32*, [7 x i8]*)

With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University

More information about the llvm-dev mailing list