necipfazil added a comment.

@morehouse PTAL



================
Comment at: llvm/include/llvm/IR/LLVMContext.h:97
     OB_clang_arc_attachedcall = 6, // "clang.arc.attachedcall"
+    OB_type = 7,                   // "type"
   };
----------------
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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105909

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to