[PATCH] D12347: [MC/AsmParser] Avoid setting MCSymbol.IsUsed

Saleem Abdulrasool via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 25 20:59:19 PDT 2015


compnerd added a subscriber: compnerd.
compnerd added a comment.

The various `false` parameters slightly muddle the code IMO, since they are for value accessors.


================
Comment at: test/MC/AsmParser/reassign.s:1
@@ +1,2 @@
+// RUN: llvm-mc -triple i386-unknown-unknown %s
+
----------------
Can you add a CHECK, or in your case, a CHECK-NOT here please?


http://reviews.llvm.org/D12347





More information about the llvm-commits mailing list