[PATCH] D12457: [Bitcode][Asm] Teach LLVM to read and write operand bundles.

[Joseph Tremoulet via llvm-commits]

JosephTremoulet added inline comments.

================
Comment at: lib/AsmParser/LLParser.cpp:1954
@@ +1953,3 @@
+
+    auto ParseOperandBundle = [&]() {
+      std::string Tag;
----------------
Why use a lambda here as opposed to just inlining it?

================
Comment at: test/Bitcode/operand-bundles.ll:19
@@ +18,3 @@
+ entry:
+; The operand bundle set [] is equivalent to "no operand bundles"
+  call void @callee0() [ ]
----------------
Do we really want two ways to represent that?  I think reporting an error for [ ] would be appropriate.


http://reviews.llvm.org/D12457