[PATCH] D66677: [GlobalISel] Introduce a G_DYN_STACKALLOC opcode to represent dynamic allocas
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 23 12:54:08 PDT 2019
arsenm added inline comments.
================
Comment at: llvm/include/llvm/Target/GenericOpcodes.td:195
+ let InOperandList = (ins type1:$size, i32imm:$align);
+ let hasSideEffects = 0;
+}
----------------
Should this be considered a side effect?
================
Comment at: llvm/test/MachineVerifier/test_g_dyn_stackalloc.mir:2
+# RUN: not llc -march=aarch64 -o /dev/null -run-pass=none -verify-machineinstrs %s 2>&1 | FileCheck %s
+# REQUIRES: global-isel, aarch64-registered-target
+
----------------
REQUIRES: global-isel isn't necessary anymore
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