This is an archive of the discontinued LLVM Phabricator instance.

[LLVM][CMake][NFC] Resolve FIXME: Rename LLVM_CMAKE_PATH to LLVM_CMAKE_DIR throughout the project
ClosedPublic

Authored by gAlfonso-bit on Aug 8 2021, 10:10 AM.

Details

Reviewers
MaskRay
ldionne
Group Reviewers
Restricted Project
Restricted Project
Commits
rGa2c319fdc6b3: [LLVM][CMake][NFC] Resolve FIXME: Rename LLVM_CMAKE_PATH to LLVM_CMAKE_DIR…
Summary

This way, we do not need to set LLVM_CMAKE_PATH to LLVM_CMAKE_DIR when (NOT LLVM_CONFIG_FOUND)

Diff Detail

Event Timeline

gAlfonso-bit created this revision.Aug 8 2021, 10:10 AM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
gAlfonso-bit requested review of this revision.Aug 8 2021, 10:10 AM
Herald added projects: Restricted Project, Restricted Project, Restricted Project, Restricted Project. · View Herald TranscriptAug 8 2021, 10:10 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added subscribers: llvm-commits, lldb-commits, Restricted Project, cfe-commits. · View Herald Transcript
gAlfonso-bit retitled this revision from [NFC] Resolve FIXME: Rename LLVM_CMAKE_PATH to LLVM_CMAKE_DIR throughout the project to [LLVM][CMake][NFC] Resolve FIXME: Rename LLVM_CMAKE_PATH to LLVM_CMAKE_DIR throughout the project.Aug 8 2021, 4:04 PM
ldionne accepted this revision.Aug 9 2021, 8:36 AM
ldionne added a subscriber: ldionne.

This LGTM.

gAlfonso-bit removed a reviewer: Restricted Project.Aug 9 2021, 12:33 PM
gAlfonso-bit removed a project: Restricted Project.
This revision is now accepted and ready to land.Aug 9 2021, 12:33 PM

Alright. Thank you!

This comment was removed by gAlfonso-bit.
This comment was removed by gAlfonso-bit.

@ldionne is this ok? any other changes?

This comment was removed by gAlfonso-bit.
This comment was removed by gAlfonso-bit.

Rebased to main

@ldionne can we land this please?

I don't think there are any other outstanding issues. Are we good to accept and merge? @ldionne

gAlfonso-bit updated this revision to Diff 370957.EditedSep 6 2021, 12:22 PM

Rebased to Main @ldionne

Since @ldionne approved this patch it's good to land. If you don't have commit access, can you provide "Your name" <your@email.address>, then somebody can commit the change for you.

@Mordante Alfonso Gregory gfunni234@gmail.com

Rebased to main

@ldionne can you please please merge

Herald added a project: Restricted Project. · View Herald TranscriptSep 16 2021, 9:30 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript