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

Wed May 27 19:07:11 PDT 2020

skan marked an inline comment as done.
skan added inline comments.

================
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"
----------------
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
```

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80289