[all-commits] [llvm/llvm-project] 258ecf: [HIP] do not use -mconstructor-aliases for device

[Yaxun (Sam) Liu via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 258ecf5f33e7b0b43bc1c7252cd84eea596c39d2
      https://github.com/llvm/llvm-project/commit/258ecf5f33e7b0b43bc1c7252cd84eea596c39d2
  Author: Yaxun (Sam) Liu <yaxun.liu at amd.com>
  Date:   2021-03-05 (Fri, 05 Mar 2021)

  Changed paths:
    M clang/lib/Driver/ToolChains/Clang.cpp
    M clang/test/Driver/hip-options.hip

  Log Message:
  -----------
  [HIP] do not use -mconstructor-aliases for device

Like nvptx and some other targets, -mconstructor-aliases does not work well with amdgpu,
therefore we disable it in the same approach.

Reviewed by: Artem Belevich

Differential Revision: https://reviews.llvm.org/D97959


  Commit: 5b3fc7180c8e4a2c57946d5e3cc325744a770717
      https://github.com/llvm/llvm-project/commit/5b3fc7180c8e4a2c57946d5e3cc325744a770717
  Author: Yaxun (Sam) Liu <yaxun.liu at amd.com>
  Date:   2021-03-05 (Fri, 05 Mar 2021)

  Changed paths:
    M clang/lib/Driver/ToolChains/Clang.cpp
    M clang/test/Driver/hip-options.hip

  Log Message:
  -----------
  [HIP] do not use -munsafe-fp-atomics by default

A bug was introduced when adding -munsafe-fp-atomics.
By default it should be off.

Reviewed by: Artem Belevich

Differential Revision: https://reviews.llvm.org/D97967


Compare: https://github.com/llvm/llvm-project/compare/f0686569cc57...5b3fc7180c8e