This is an archive of the discontinued LLVM Phabricator instance.

[Flang] Fixed installation permission of the "binary" flang
ClosedPublic

Authored by tianshilei1992 on Sep 16 2020, 12:04 PM.

Details

Summary

Under current configuration, the permission of flang after installation is 700.
This could bring a problem for system administrators who build and install flang
for other users, which only the user who builds LLVM can execute it, and others
can not. In this patch, the explicit permission setting in the install command
is removed, and let CMake determine what perssion to be used like other components.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald Transcript
tianshilei1992 requested review of this revision.Sep 16 2020, 12:04 PM
This revision is now accepted and ready to land.Sep 16 2020, 2:55 PM

Rebase and trigger a new test

This revision was landed with ongoing or failed builds.Sep 16 2020, 3:54 PM
This revision was automatically updated to reflect the committed changes.