[PATCH] D75218: Make asm printer for X86 use custom separator instead of hard-coded tabs.

[Clement Courbet via Phabricator via llvm-commits]

courbet added inline comments.


================
Comment at: llvm/include/llvm/MC/MCInstPrinter.h:62
 
+  /// Separator string, canonically "\t".
+  std::string Separator = "\t";
----------------
Given that this is only used by the X86 backend, why is it in the target-independent abstraction ?
I think it would make sense to either:
 - Keep this here, but keep the behaviour consistent between all targets (e.g. AArch64 also uses '\t' as its separator). You might even be able to get rid of `supportsCustomSeparator`
 - Move this to X86, and replace `virtual bool supportCustomSeparator()` + `void setSeparator()` by   `virtual bool setSeparator()`. If more targets start opting in, then the code can be moved back to the target-independent abstraction.


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

https://reviews.llvm.org/D75218