[all-commits] [llvm/llvm-project] b8c16d: [lldb][TypeSystemClang][NFCI] Use LangOptions::set...

Michael Buch via All-commits all-commits at lists.llvm.org
Mon Apr 15 13:45:46 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b8c16db938327efdabc63f21fdc770069ac4406b
      https://github.com/llvm/llvm-project/commit/b8c16db938327efdabc63f21fdc770069ac4406b
  Author: Michael Buch <michaelbuch12 at gmail.com>
  Date:   2024-04-15 (Mon, 15 Apr 2024)

  Changed paths:
    M lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp

  Log Message:
  -----------
  [lldb][TypeSystemClang][NFCI] Use LangOptions::setLangDefaults when creating new LangOptions (#88721)

This logic was originally copied from `CompilerInstance::parseLangArgs`.
Since then, the `CompilerInstance` uses `LangOptions::setLangDefaults`
to set up new `LangOptions` instances. In our case, we only ever passed
`Language::ObjCXX` into LLDB's `ParseLangArgs`, so most of this function
was dead code.

This patch replaces the duplicated logic with a call to
`LangOptions::setLangDefaults`.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list