[all-commits] [llvm/llvm-project] 878097: [APINotes] Fix failing tests after a PCM logic change

Egor Zhdan via All-commits all-commits at lists.llvm.org
Mon Mar 11 05:02:51 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 878097dff3ea4bad6b7f50017224a84bbf2af406
      https://github.com/llvm/llvm-project/commit/878097dff3ea4bad6b7f50017224a84bbf2af406
  Author: Egor Zhdan <e_zhdan at apple.com>
  Date:   2024-03-11 (Mon, 11 Mar 2024)

  Changed paths:
    M clang/lib/APINotes/APINotesManager.cpp

  Log Message:
  -----------
  [APINotes] Fix failing tests after a PCM logic change

This fixes tests that are going to be upstreamed in the near future.
Currently they are failing downstream in the Apple open source fork.

Failing tests
  Clang :: APINotes/retain-count-convention.m
  Clang :: APINotes/types.m
  Clang :: APINotes/versioned-multi.c
  Clang :: APINotes/versioned.m

Since 2e5af56 got merged, Clang now enables `LangOpts.APINotesModules`
when reading a precompiled module that was built with API Notes enabled.
This is correct. The logic in APINotesManager needs to be adjusted to
handle this.

rdar://123526142



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