[all-commits] [llvm/llvm-project] 6c745e: [Attributor][FIX] Ensure order for multiple refere...

[Johannes Doerfert via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6c745e04fac8241e1be6ef90a654228d4eb3ff2b
      https://github.com/llvm/llvm-project/commit/6c745e04fac8241e1be6ef90a654228d4eb3ff2b
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2022-01-08 (Sat, 08 Jan 2022)

  Changed paths:
    M llvm/lib/Transforms/IPO/AttributorAttributes.cpp

  Log Message:
  -----------
  [Attributor][FIX] Ensure order for multiple references into map

If we have multiple references into a map we need to ensure the ones
created late do not invalidate the ones created early. To do that we
need to make sure all but the first are not modifying the map, hence
for them the keys have to be present already.

Fixes #52875.


  Commit: 37639b72a17be443e7f73e906554f8bbd0dbf271
      https://github.com/llvm/llvm-project/commit/37639b72a17be443e7f73e906554f8bbd0dbf271
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2022-01-08 (Sat, 08 Jan 2022)

  Changed paths:
    M clang/lib/CodeGen/CGStmtOpenMP.cpp
    M clang/test/OpenMP/debug_private.c
    M clang/test/OpenMP/debug_task_shared.c

  Log Message:
  -----------
  [OpenMP][FIX] Emit debug declares only if debug info is available

The `EmitDeclareOfAutoVariable` introduced in D114504 and D115510 has a
precondition that cannot be violated. It is unclear if we should call it
directly given the sparse usage in clang but for now we should at least
not crash if the debug info kind is too low.

Fixes #52938.

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


Compare: https://github.com/llvm/llvm-project/compare/6a10bc705681...37639b72a17b