[PATCH] D125699: [StackColoring] Don't merge slots with differing StackIDs

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added inline comments.


================
Comment at: llvm/lib/CodeGen/StackColoring.cpp:1322
+        // Objects with different stack IDs cannot be merged.
+        if (MFI->getStackID(SortedSlots[I]) != MFI->getStackID(SortedSlots[J]))
+          continue;
----------------
frasercrmck wrote:
> sdesmalen wrote:
> > minor nit: if you move this code two lines below, you can use `FirstSlot` and `SecondSlot`.
> Ach of course, thank you. And I'm glad you got in touch as I woke up this morning feeling guilty I hadn't notified any AArch64 people when this might (and indeed does) affect them :)
No worries, I've got a Herald rule set up for patches related to scalable vectors, hence why I spotted it. I appreciate the (after)thought though :)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125699/new/

https://reviews.llvm.org/D125699