[PATCH] D51664: [IR] Lazily number instructions for local dominance queries

[Quentin Colombet via Phabricator via llvm-commits]

qcolombet added a comment.
Herald added a project: LLVM.

Hi Reid,

Thanks for doing this.

Instead of changing the instruction and basic block classes, could we instead provide an enhanced version of ilist that does that?
E.g., something based on https://scholar.google.com/scholar?cluster=5225046542682967685&hl=en&as_sdt=0,5 (and we can add laziness on top if we want)

The rationale is that I wanted to do something similar in the Machine representation as well (and rework the SlotIndexes in the process) and was thinking that an improved version of ilist would do that for us while allowing to share the code in the middle-end and backend.

Bottom line, I was hoping this work would solve the dominance problem instead of basic block for the backend as well :).

Cheers,
-Quentin


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

https://reviews.llvm.org/D51664