[PATCH] D80289: [Driver][X86] Support branch align options with LTO

[Fangrui Song via Phabricator via cfe-commits]

MaskRay accepted this revision.
MaskRay added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/test/Driver/x86-malign-branch.c:6
 // BOUNDARY: "-mllvm" "-x86-align-branch-boundary=16"
+// RUN: %clang -target x86_64-unknown-linux -malign-branch-boundary=16 -flto %s -### 2>&1 | FileCheck %s --check-prefix=BOUNDARY-LTO
+// BOUNDARY-LTO: "-plugin-opt=-x86-align-branch-boundary=16"
----------------
skan wrote:
> MaskRay wrote:
> > Drop `-unknown-linux`
> > 
> > ditto below
> Sorry it seems I didn't submit my comment...
> 
> Dropping `-unknown-linux`  here would get an error
> 
> ```
>  error: 'x86_64': unable to pass LLVM bit-code files to linker
> ```
Ah, you are right. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80289/new/

https://reviews.llvm.org/D80289