[PATCH] D106701: [clang] Add -falign-loops=N where N is a power of 2

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 23 15:50:54 PDT 2021


MaskRay added a comment.

In D106701#2901639 <https://reviews.llvm.org/D106701#2901639>, @efriedma wrote:

> Can we hook this up to a LLVM IR function attribute, instead of making it a codegen flag?

The current TargetLoweringBase::PrefLoopAlignment is global. I have considered a function attribute, but it seems overkill for now.
(Inlining behavior is a bit unclear.)
The current use cases just need a global value instead of a refined per-function value.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106701



More information about the cfe-commits mailing list