tatyana-krasnukha added inline comments.

================
Comment at: source/Plugins/Architecture/Arc/ArchitectureArc.cpp:29
+  PluginManager::RegisterPlugin(GetPluginNameStatic(),
+                                "ARC-specific algorithms",
+                                &ArchitectureArc::Create);
----------------
clayborg wrote:
> Not a great human readable architecture name here. All other plug-ins use the 
> short architecture name ("arm", "mipc", "ppc64"). Best to just use "arc"?
You are writing about GetPluginNameStatic. The description string is same as 
for other plug-ins (just copy-pasted).


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D55718



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to