[PATCH] D105909: [clang][CallGraphSection] Add type id metadata to indirect call and targets
Matt Morehouse via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 22 10:43:32 PDT 2021
morehouse added inline comments.
================
Comment at: llvm/include/llvm/IR/LLVMContext.h:97
OB_clang_arc_attachedcall = 6, // "clang.arc.attachedcall"
+ OB_type = 7, // "type"
};
----------------
necipfazil wrote:
> morehouse wrote:
> > Do we need to update `LLVMContext::LLVMContext()`?
> Operand bundles are originally used as input to lowering in `SelectionDAGBuilder`. Arbitrary operand bundles are not allowed.
>
> To allow `type` operand bundle, we need this change. Please see `llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp` for related changes.
So why don't you need to update `LLVMContext::LLVMContext()` then? All other operand bundle types are there.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105909/new/
https://reviews.llvm.org/D105909
More information about the llvm-commits
mailing list