[PATCH] D74813: [RFC] Add hash of block contents to function block names

[Vedant Kumar via Phabricator via cfe-commits]

vsk edited reviewers, added: ahatanak, erik.pilkington, dexonsmith; removed: vsk.
vsk added a comment.

At a high level the idea sounds good to me. For out OSes, symbol name instability results in serious performance regressions as this invalidates text/data orderfiles. Why shouldn’t this be on by default?

Also, could you add tests to cover this change?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74813