davide added inline comments.

================
Comment at: source/Plugins/ABI/SysV-x86_64/ABISysV_x86_64.cpp:1916
+  std::string Name = std::string(reg_info->name);
+  bool IsCalleeSaved = llvm::StringSwitch<bool>(Name)
+    .Cases("r12", "r13", "r14", "r15",
----------------
Currently this uses two cases as the underlying `StringSwitch` implementation 
allows up to 10 arguments. I have plans to switch it to use variadic templates 
so that this code can be rewritten to use a single case.


https://reviews.llvm.org/D37420



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
  • [Lldb-commits] [PATCH]... Davide Italiano via Phabricator via lldb-commits
    • [Lldb-commits] [P... Davide Italiano via Phabricator via lldb-commits
    • [Lldb-commits] [P... Saleem Abdulrasool via Phabricator via lldb-commits
    • [Lldb-commits] [P... Davide Italiano via Phabricator via lldb-commits
      • Re: [Lldb-com... Jason Molenda via lldb-commits
        • Re: [Lldb... Davide Italiano via lldb-commits
          • Re: [... Jason Molenda via lldb-commits
            • ... Davide Italiano via lldb-commits
              • ... Greg Clayton via lldb-commits
                • ... Jason Molenda via lldb-commits
                • ... Greg Clayton via lldb-commits
                • ... Zachary Turner via lldb-commits

Reply via email to