[PATCH] D42183: [LLD] [ELF] Pass CPU string to LTO pipeline.

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 30 10:18:36 PST 2018


Craig Topper via Phabricator via llvm-commits
<llvm-commits at lists.llvm.org> writes:

> craig.topper added a comment.
>
> As of r322227 I would have though the target-cpu attribute would control the NOPL creation and override the CPU string.

Doesn't look like it.

>From the LLD point of view I think the patch is OK one way or the
other. As long as lto::Config has a CPU field it is reasonable for lld
to set it.

I will update the test to not include the target-cpu attribute for now
and commit.

Cheers,
Rafael


More information about the llvm-commits mailing list