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

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat May 23 11:39:39 PDT 2020


MaskRay added a comment.

In D80289#2051947 <https://reviews.llvm.org/D80289#2051947>, @skan wrote:

> Ping.


`https://reviews.llvm.org/D80168#2046093`

"Unless something is urgent, the usual practice is to ping after a week rather than 24 hours, as many people have big piles on their review plate, so can't always get to it the next day."



================
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"
----------------
Drop `-unknown-linux`

ditto below


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80289





More information about the cfe-commits mailing list