tree 120a0cdf8c5c7ffc93319104f337847ec866fc01
parent 6ddf7cf7808e7e60314ed003d52c215dedc0924b
author Matt Arsenault <Matthew.Arsenault@amd.com> 1743699145 +0700
committer GitHub <noreply@github.com> 1743699145 +0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJn7rzJCRC1aQ7uu5UhlAAAmm4QACSUbTB2vWou3nNvZiiWJ5vd
 4NN+FKMN3WiRpwmbzZP84Ewg0jTyrc9S1dHm1YxJxkZiYzD8cICAe5F+j9/oPCcj
 UKbeKOCdeO5LBZKpyi0pUKzX3QrolihnKVOSjz6z5IFvdkNwtW3VulRvZhJryTmN
 KHUFsA8jtqBm8Fo0KgvUk12E1dxXxFasJcFQ9xrH18iA/sBGbkH9iwObYPASWRhQ
 5s+5E2Dx6m8mD9s5RihRtqZ6sLhxLpDfJ41nVbOM9J8nSo7L90+4Z4qGFjUSGJbq
 IRmSEYVXIr/1IiL/0VKS1wiJTrYYJD/PF0AdnXM3KtuahtW6pvCgJS13kHx23ds1
 rL7V8wIVOn/N/nvobEw92xxR6YMKDaiTtlzJj9J+eHTiYD8UXmpavMs42AiWogOk
 f/PCA/+lo2YPd22GOCfjPBB4mSWafBdsiSpgmaekTNU7u9O9iDtdULdYfRXgbLU9
 nHwpKZaxBbahtTND0Ehixi/tzH/CrYLw9prQ14fKlEUVCv0MQ5TPntRTrSEacKA+
 r97z6GNJO1e9uAPxakaj3pTHHuMdN/Spj4Kr3F88LdgLprYcxnR19PPx5S1CbVai
 qhH1FWfUZony4xZZnrh/a2pYopFWsNmUhCkS+MPbhnGfSmsbiQhHz5JcXkV47kNo
 QJc4oqej2EdlJmxuX3WM
 =v7vV
 -----END PGP SIGNATURE-----
 

CloneFunction: Do not delete blocks with address taken (#134209)

If a block with a single predecessor also had its address taken,
it was getting deleted in this post-inline cleanup step. This would
result in the blockaddress in the resulting function getting deleted
and replaced with inttoptr 1.

This fixes one bug required to permit inlining of functions with blockaddress 
uses.

At the moment this is not testable (at least without an annoyingly complex
unit test),  and is a pre-bug fix for future patches. Functions with
blockaddress uses are rejected in isInlineViable, so we don't get this far
with the current InlineFunction uses (some of the existing cases seem to
reproduce this part of the rejection logic, like PartialInliner). This
will be tested in a pending llvm-reduce change.

Prerequisite for #38908