[Openmp-commits] [PATCH] D47224: [cmake] Guard another instance where symlinks are being created

[Azharuddin Mohammed via Phabricator via Openmp-commits]

azharudd added a comment.

In https://reviews.llvm.org/D47224#1112505, @Hahnfeld wrote:

> I'm not sure about this patch: These commands do not install the aliases, they are created in the build directory and may be needed for testing.


This should be NFC as by default that flag would be ON. I see it being used in lines 306-314 which is dealing with installing the symlinks. Maybe this is just duplicate code.

> (And shouldn't this say `if (LIBOMP_INSTALL_ALIASES)`, according to your change in https://reviews.llvm.org/D47221?)

I just reused the existing if condition in this file and wanted to remain consistent. At least it doesn't cause a cmake error in this case as the if condition would still be correct.


Repository:
  rL LLVM

https://reviews.llvm.org/D47224