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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
