[all-commits] [llvm/llvm-project] 7e928f: [llvm] fix missing and incorrect LLVM_ABI annotati...

[Andrew Rogers via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7e928f9d37788a8143c7704c98133f107612b87d
      https://github.com/llvm/llvm-project/commit/7e928f9d37788a8143c7704c98133f107612b87d
  Author: Andrew Rogers <andrurogerz at gmail.com>
  Date:   2025-07-08 (Tue, 08 Jul 2025)

  Changed paths:
    M llvm/include/llvm/Analysis/FunctionPropertiesAnalysis.h
    M llvm/include/llvm/Analysis/IR2Vec.h
    M llvm/include/llvm/Analysis/TargetLibraryInfo.h
    M llvm/include/llvm/CodeGen/Passes.h
    M llvm/include/llvm/DebugInfo/DWARF/DWARFDataExtractor.h
    M llvm/include/llvm/DebugInfo/DWARF/LowLevel/DWARFCFIProgram.h
    M llvm/include/llvm/DebugInfo/DWARF/LowLevel/DWARFDataExtractorSimple.h
    M llvm/include/llvm/Frontend/OpenMP/DirectiveNameParser.h
    M llvm/include/llvm/IR/DebugInfoMetadata.h
    M llvm/include/llvm/IR/IRBuilder.h
    M llvm/include/llvm/IR/ProfDataUtils.h
    M llvm/include/llvm/IR/RuntimeLibcalls.h
    M llvm/include/llvm/MC/MCContext.h
    M llvm/include/llvm/MC/MCFixup.h
    M llvm/include/llvm/MC/MCSection.h
    M llvm/include/llvm/MC/MCSectionGOFF.h
    M llvm/include/llvm/Transforms/IPO/Attributor.h

  Log Message:
  -----------
  [llvm] fix missing and incorrect LLVM_ABI annotations (#147399)

## Purpose

This patch is one in a series of code-mods that annotate LLVM’s public
interface for export. This patch adds `LLVM_ABI` annotations to some
recently added symbols that were previously missed. It also removes
`LLVM_ABI` from symbols where it is problematic. The LLVM annotation
currently has no meaningful impact on the LLVM build; however, it is a
prerequisite to support an LLVM Windows DLL (shared library) build.

## Overview

These changes were generated automatically using the [Interface
Definition Scanner (IDS)](https://github.com/compnerd/ids) tool,
followed formatting with `git clang-format`.

NOTE: I am using a modified version of IDS with local patches so it
removes unnecessary export annotations. These local IDS changes
generated all of the `LLVM_ABI` deletions in this PR.

## Background

This effort is tracked in #109483. Additional context is provided in
[this
discourse](https://discourse.llvm.org/t/psa-annotating-llvm-public-interface/85307),
and documentation for `LLVM_ABI` and related annotations is found in the
LLVM repo
[here](https://github.com/llvm/llvm-project/blob/main/llvm/docs/InterfaceExportAnnotations.rst).


## Validation

Local builds and tests to validate cross-platform compatibility. This
included llvm, clang, and lldb on the following configurations:

- Windows with MSVC
- Windows with Clang
- Linux with GCC
- Linux with Clang
- Darwin with Clang



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