This is an archive of the discontinued LLVM Phabricator instance.

[clang][LTO] Remove the use of `--` for arange option
ClosedPublic

Authored by qiongsiwu1 on Oct 6 2022, 2:04 PM.

Details

Summary

https://reviews.llvm.org/D134668 removed all -- (double dashes) when using plugin-opt to pass linker options and replaced them with -. https://reviews.llvm.org/D133092 was committed later but introduced an instance of --. This patch replaces the -- with -.

Diff Detail

Event Timeline

qiongsiwu1 created this revision.Oct 6 2022, 2:04 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 6 2022, 2:04 PM
qiongsiwu1 requested review of this revision.Oct 6 2022, 2:04 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 6 2022, 2:04 PM
qiongsiwu1 updated this revision to Diff 465872.Oct 6 2022, 2:06 PM

Fix a patch upload error.

qiongsiwu1 removed a project: Restricted Project.Oct 6 2022, 2:06 PM
qiongsiwu1 edited the summary of this revision. (Show Details)

I missed https://reviews.llvm.org/D134668 . See my comment there I don't think the change is necessary.

I missed https://reviews.llvm.org/D134668 . See my comment there I don't think the change is necessary.

Thanks for the comment! I posted a reply there.

MaskRay accepted this revision.Oct 6 2022, 3:22 PM
MaskRay added inline comments.
clang/test/Driver/debug-options-aranges.c
6
This revision is now accepted and ready to land.Oct 6 2022, 3:22 PM
qiongsiwu1 updated this revision to Diff 465910.Oct 6 2022, 3:36 PM

Thanks for the review! Comment addressed.

qiongsiwu1 marked an inline comment as done.Oct 6 2022, 3:36 PM