[PATCH] D125388: Allow type-mismatching RAUW of values in metadata, and add C API.

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 20 01:54:45 PDT 2022


nikic requested changes to this revision.
nikic added a comment.
This revision now requires changes to proceed.

I don't think this change makes sense on a couple of levels:

- I don't believe that `ValueAsMetadata::handleRAUW()` is intended as a public API at all.
- The original problem seems like a bug in GlobalDCE -- did anyone investigate why it happens?
- With opaque pointers the replacement doesn't use a bitcast anyway, so the issue is "fixed" as a side-effect. As this API will only go into LLVM 15, it would not be useful.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125388



More information about the llvm-commits mailing list