[all-commits] [llvm/llvm-project] d3b75c: [clang] Make `-fvisibility={}` and `-ftype-visibil...

Juergen Ributzka via All-commits all-commits at lists.llvm.org
Thu Nov 16 08:41:33 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d3b75c4750673b6d0d8a224f02b2c0885209a49a
      https://github.com/llvm/llvm-project/commit/d3b75c4750673b6d0d8a224f02b2c0885209a49a
  Author: Juergen Ributzka <juergen at ributzka.de>
  Date:   2023-11-16 (Thu, 16 Nov 2023)

  Changed paths:
    M clang/include/clang/Basic/LangOptions.def
    A clang/test/ClangScanDeps/strip-visibility.c
    A clang/test/Modules/codegen-visibility.cpp
    A clang/test/Modules/visibility.cpp

  Log Message:
  -----------
  [clang] Make `-fvisibility={}` and `-ftype-visibility={}` benign options. (#71985)

Both options do not affect the AST content that is serialized into the PCM. This
commit includes the following changes:
    
1.) Mark `-fvisibility={}` and `-ftype-visibility={}` as benign options.That
     means they are no longer considered part of the module hash, which can
     reduce the number of module variants.
    
2.) Add a test to verify the generated LLVM IR is not affected by the default
     visibiliy mode in the module.

3.) Add a test to clang-scan-deps to ensure only one module is build, even if
      the above mentioned options are used.
    
This fixes rdar://118246054.




More information about the All-commits mailing list