This is an archive of the discontinued LLVM Phabricator instance.

Differential D96484

[flang] Fix typo in FlangConfig.cmake.in.
ClosedPublic

Authored by leandrov on Feb 11 2021, 3:18 AM.

Details

Reviewers
sscalpone
thopre
Commits
rG204360fd71d6: [flang] Fix typo in FlangConfig.cmake.in.
Summary

find_package(Flang) does not work as there is a missing @ in the
FlangConfig.cmake.in file. This patch fixes the issue.

Diff Detail

Event Timeline

leandrov created this revision.Feb 11 2021, 3:18 AM
Herald added a reviewer: sscalpone. · View Herald TranscriptFeb 11 2021, 3:18 AM
Herald added a subscriber: mgorny. · View Herald Transcript
leandrov requested review of this revision.Feb 11 2021, 3:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 11 2021, 3:18 AM
Herald added a subscriber: llvm-commits. · View Herald Transcript
thopre accepted this revision.Feb 11 2021, 3:20 AM
thopre added a subscriber: thopre.

LGTM

This revision is now accepted and ready to land.Feb 11 2021, 3:20 AM
Harbormaster completed remote builds in B88779: Diff 322938.Feb 11 2021, 4:28 AM
leandrov updated this revision to Diff 322986.Feb 11 2021, 6:16 AM

Rebase.

Harbormaster completed remote builds in B88800: Diff 322986.Feb 11 2021, 7:10 AM
Closed by commit rG204360fd71d6: [flang] Fix typo in FlangConfig.cmake.in. (authored by leandrov, committed by thopre). · Explain WhyFeb 11 2021, 10:09 AM
This revision was automatically updated to reflect the committed changes.
thopre added a commit: rG204360fd71d6: [flang] Fix typo in FlangConfig.cmake.in..

Revision Contents

PathSize
flang/
cmake/
modules/
2 lines

Diff 322938

flang/cmake/modules/FlangConfig.cmake.in

# This file allows users to call find_package(Flang) and pick up our targets. # This file allows users to call find_package(Flang) and pick up our targets.
@FLANG_CONFIG_CODE@ @FLANG_CONFIG_CODE@
find_package(LLVM REQUIRED CONFIG find_package(LLVM REQUIRED CONFIG
HINTS "@FLANG_CONFIG_LLVM_CMAKE_DIR@") HINTS "@FLANG_CONFIG_LLVM_CMAKE_DIR@")
set(FLANG_EXPORTED_TARGETS "@FLANG_EXPORTS@") set(FLANG_EXPORTED_TARGETS "@FLANG_EXPORTS@")
set(FLANG_CMAKE_DIR "FLANG_CONFIG_CMAKE_DIR@") set(FLANG_CMAKE_DIR "@FLANG_CONFIG_CMAKE_DIR@")
set(FLANG_INCLUDE_DIRS "@FLANG_CONFIG_INCLUDE_DIRS@") set(FLANG_INCLUDE_DIRS "@FLANG_CONFIG_INCLUDE_DIRS@")
# Provide all our library targets to users. # Provide all our library targets to users.
include("@FLANG_CONFIG_EXPORTS_FILE@") include("@FLANG_CONFIG_EXPORTS_FILE@")