[PATCH] D66677: [GlobalISel] Introduce a G_DYN_STACKALLOC opcode to represent dynamic allocas
Amara Emerson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 23 13:40:30 PDT 2019
aemerson marked an inline comment as done.
aemerson added inline comments.
================
Comment at: llvm/include/llvm/Target/GenericOpcodes.td:195
+ let InOperandList = (ins type1:$size, i32imm:$align);
+ let hasSideEffects = 0;
+}
----------------
arsenm wrote:
> aemerson wrote:
> > arsenm wrote:
> > > Should this be considered a side effect?
> > I don't think so? G_FRAME_INDEX doesn't have side effects, so I don't think this should be counted as having it either. I do intend to use this to implement stack checking calls but that's a target dependent thing.
> The DAG equivalent has a chain. I think you could observe different behavior if you reordered a small and a large request
That's true. I'll change it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66677/new/
https://reviews.llvm.org/D66677
More information about the llvm-commits
mailing list