[patch] set AssemblerDialect

Project Infrastructure LLVM Dev List Archives
#1

currently, there is no (easy) way to set the AssemblerDialect. the
only method i am aware of is to set that via cl:opt.

this patch fixes that by adding a public function
setAssemblerDialect() to class MCAsmInfo.

Signed-off-by: Jun Koi <junkoi2004@gmail.com>

diff --git a/include/llvm/MC/MCAsmInfo.h b/include/llvm/MC/MCAsmInfo.h
index 97aad71..cd08a7e 100644
--- a/include/llvm/MC/MCAsmInfo.h
+++ b/include/llvm/MC/MCAsmInfo.h
@@ -453,6 +453,9 @@ namespace llvm {
     unsigned getAssemblerDialect() const {
       return AssemblerDialect;
     }
+ void setAssemblerDialect(unsigned dialect) {
+ AssemblerDialect = dialect;
+ }
     bool doesAllowQuotesInName() const {
       return AllowQuotesInName;
     }

#2

What's the use case?

#3

What's the use case?

the use case is that we can print out the assembly using alternative dialect.

currently, on Intel machine, the default dialect is AT&T, and we
cannot change that to Intel syntax without using cl:opt.
the new public method lets us change the dialect without having cl:opt involved.

thanks,
Jun

#4

Sorry, I wasn't clear. At what point in the code are you looking to change this and why? This is something that, normally, should be set when the MCAsmInfo for the target is created and never changed afterwards. This patch changes that dynamic, and I'm trying to understand why that's desirable.

-jIm