[PATCH] D130436: [BOLT] Process each block only once in fixCFGForPIC

[Maksim Panchenko via Phabricator via llvm-commits]

maksfb added a comment.

Thanks for improving the efficiency of `fixCFGForPIC()`. The change looks fine to me. Do you think the implementation will be simpler if we make `Blocks` a queue and add newly added blocks to the end? This way we can get rid of the inner loop and dealing with `CurrentBB`/`NextBB`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130436