[PATCH] D32021: CodeGen: Let frame index value type match alloca addr space

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

Can you add a test using the new mapping for this



================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAG.cpp:1829
   int FrameIdx = MFI.CreateStackObject(ByteSize, StackAlign, false);
-  return getFrameIndex(FrameIdx, TLI->getPointerTy(getDataLayout()));
+  return getFrameIndex(FrameIdx, TLI->getPointerTy(getDataLayout(),
+      getDataLayout().getAllocaAddrSpace()));
----------------
Add a new TLI->getAllocaPointerTy?


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:5418
   }
   case Intrinsic::stackprotector: {
     // Emit code into the DAG to store the stack guard onto the stack.
----------------
This one hasn't been updated with mangling yet, so might as well leave it for now


https://reviews.llvm.org/D32021