[all-commits] [llvm/llvm-project] c8c583: Fix diag for read-only target features

Yaxun (Sam) Liu via All-commits all-commits at lists.llvm.org
Fri Jun 16 14:48:59 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c8c583071591dbc6927955e608dcd44c0bac8b9c
      https://github.com/llvm/llvm-project/commit/c8c583071591dbc6927955e608dcd44c0bac8b9c
  Author: Yaxun (Sam) Liu <yaxun.liu at amd.com>
  Date:   2023-06-16 (Fri, 16 Jun 2023)

  Changed paths:
    M clang/lib/Basic/TargetInfo.cpp
    M clang/lib/Basic/Targets.cpp
    A clang/test/OpenMP/openmp-read-only-feature.c

  Log Message:
  -----------
  Fix diag for read-only target features

Currently the diag is emitted even when there is no
target feature specified on command line for OpenMP.
This is because the function to initialize feature map
is also used with cached feature string. The fix is to
only diag when the feature map is initialized with
feature strings from command line options.

Reviewed by: Joseph Huber, Matt Arsenault, Johannes Doerfert

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




More information about the All-commits mailing list