[PATCH] D45445: [CodeGen/AccelTable] Add the "sizetype" synthetic type to the accelerator table

[Pavel Labath via Phabricator via llvm-commits]

labath added inline comments.


================
Comment at: lib/CodeGen/AsmPrinter/DwarfUnit.cpp:1407
   IndexTyDie = &createAndAddDIE(dwarf::DW_TAG_base_type, getUnitDie());
-  addString(*IndexTyDie, dwarf::DW_AT_name, "sizetype");
+  StringRef Name = "sizetype";
+  addString(*IndexTyDie, dwarf::DW_AT_name, Name);
----------------
Another interesting question is whether we shouldn't rename this to something more unique (`_Sizetype` ?). As it stands now, if a user has his own `sizetype` defined, lldb will get confused and will not be able to differentiate the two. Gcc emits this type as "unsigned long", but doing that is a bit trickier for us as we don't want to assume a particular front-end here.


Repository:
  rL LLVM

https://reviews.llvm.org/D45445