[PATCH] D67626: [IR] Merge metadata manipulation code into Value

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 16 10:02:53 PDT 2019


dexonsmith added a comment.

I don't think it should be possible to attach metadata to constants owned by the LLVMContext (such as `i32 0`) since that seems like a bug, but IIUC this patch is enabling that.  Is that intentional?  If so, why?  If not, can you find a way to avoid making that possible?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67626





More information about the llvm-commits mailing list