[all-commits] [llvm/llvm-project] 18e70a: [llvm][TargetParser] Return StringMap from getHost...

David Spickett via All-commits all-commits at lists.llvm.org
Thu Jul 11 02:33:05 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 18e70a4d5042299054dae7d3995f6ccd8f4112b3
      https://github.com/llvm/llvm-project/commit/18e70a4d5042299054dae7d3995f6ccd8f4112b3
  Author: David Spickett <david.spickett at linaro.org>
  Date:   2024-07-11 (Thu, 11 Jul 2024)

  Changed paths:
    M clang/lib/Driver/ToolChains/Arch/ARM.cpp
    M clang/lib/Driver/ToolChains/Arch/X86.cpp
    M lldb/utils/lit-cpuid/lit-cpuid.cpp
    M llvm/include/llvm/TargetParser/Host.h
    M llvm/lib/CodeGen/CommandFlags.cpp
    M llvm/lib/ExecutionEngine/Orc/JITTargetMachineBuilder.cpp
    M llvm/lib/Target/TargetMachineC.cpp
    M llvm/lib/TargetParser/Host.cpp

  Log Message:
  -----------
  [llvm][TargetParser] Return StringMap from getHostCPUFeatures (#97824)

Previously this took a reference to a map and returned a bool to say
whether it succeeded. We can return a StringMap instead, as all callers
but 1 simply iterated the map if the bool was true, and passed in empty
maps as the starting point.

lldb's lit-cpuid did specifically check whether the call failed, but due
to the way the x86 routines work this works out the same as checking if
the returned map is empty.



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