[all-commits] [llvm/llvm-project] 838826: [BOLT] Process each block only once in fixCFGForPIC

[Fabian Parzefall via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 83882606dbd7ffb0bdd3460356202d97705809c8
      https://github.com/llvm/llvm-project/commit/83882606dbd7ffb0bdd3460356202d97705809c8
  Author: Fabian Parzefall <parzefall at fb.com>
  Date:   2022-07-25 (Mon, 25 Jul 2022)

  Changed paths:
    M bolt/include/bolt/Core/BinaryFunction.h
    M bolt/include/bolt/Passes/ValidateInternalCalls.h
    M bolt/lib/Passes/ValidateInternalCalls.cpp

  Log Message:
  -----------
  [BOLT] Process each block only once in fixCFGForPIC

Rather than iterating over the whole function from the start until no
internal calls are found, process each block only once and continue
processing after splitting. This version of the function also does not
seemingly invalidate iterators from within the loop.

Reviewed By: maksfb

Differential Revision: https://reviews.llvm.org/D130436