[PATCH] D69136: Add a heap alloc site marker field to the ExtraInfo in MachineInstrs.

Amy Huang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 24 16:12:40 PDT 2019


akhuang added a comment.

Sounds good. I also realized that the logic for the "Should never have only a single symbol allocated out-of-line!" is wrong now. I added a parameter to the function to keep track of whether the assert should happen. Or do you think it can just be removed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69136





More information about the llvm-commits mailing list