[all-commits] [llvm/llvm-project] ab8abe: [Driver] Change CLANG_ENABLE_OPAQUE_POINTERS_INTER...

Fangrui Song via All-commits all-commits at lists.llvm.org
Wed Apr 13 16:58:13 PDT 2022

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ab8abeaf48abb3addcbdb6136b1975b4afa3b37c
  Author: Fangrui Song <i at maskray.me>
  Date:   2022-04-13 (Wed, 13 Apr 2022)

  Changed paths:
    M clang/include/clang/Driver/Options.td
    M clang/lib/Driver/ToolChains/Clang.cpp
    M clang/test/CMakeLists.txt
    A clang/test/Driver/opaque-pointers-off.c
    M clang/test/Driver/rewrite-legacy-objc.m
    M clang/test/Driver/rewrite-objc.m
    M clang/test/lit.cfg.py
    M clang/test/lit.site.cfg.py.in

  Log Message:
  [Driver] Change CLANG_ENABLE_OPAQUE_POINTERS_INTERNAL to affect driver default instead of cc1 default

The current cc1 CLANG_ENABLE_OPAQUE_POINTERS=on default difference is not ideal
in that people contribute %clang_cc1 tests may assume the default ON behavior,
which will cause failures on systems set to OFF.

cc1 option default dependent on CMake options should be used prudently
(generally avoided). We prefer to limit target differences to Driver.

Change the CLANG_ENABLE_OPAQUE_POINTERS_INTERNAL mechanism introduced in D123122
to use a driver default instead. This is similar to the mechanism used for the
-flegacy-pass-manager transition to new PM transition.

Reviewed By: #opaque-pointers, rsmith, aeubanks

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

More information about the All-commits mailing list