[PATCH] D20337: [MC] Support symbolic expressions in assembly directives

[Petr Hosek via llvm-commits]

phosek added inline comments.

================
Comment at: include/llvm/MC/MCStreamer.h:602
@@ +601,3 @@
+  /// \param Loc - The location of the expression for error reporting.
+  virtual void emitRepeatedValue(const MCExpr *NumValues, int64_t Size,
+                                 int64_t Expr, SMLoc Loc = SMLoc());
----------------
niravd wrote:
> Why isn't this named EmitFill? It's of a distinct type from the other calls and is only for dealing with Fill directives and the distinction seems more confusing than multiple variants of EmitFill. 
`EmitFill` is used for emitting "fill bytes" in case of directives like `.space` and `.zero` while `.fill` requires a special handling. I could rename both `emitSpace` and `emitRepeatedValue` to `EmitFill` if you prefer since it's distinct type from the other one, I just wanted to avoid confusion, but I feel like it's going to be confusing in either case.


Repository:
  rL LLVM

http://reviews.llvm.org/D20337