[PATCH] D134282: [CGP] Add generic TargetLowering::shouldAlignPointerArgs() implementation

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 23 12:20:30 PST 2023


efriedma reopened this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

Is there any way we can make AIX targets more resilient handling this kind of alignment change?  If we have to, we can choose a point after which passes aren't allowed to increase the alignment of globals, and move this transform before that.  But I'd like to avoid that if possible.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134282



More information about the llvm-commits mailing list