[PATCH] D67914: [Alignment][NFC] DataLayout migration to llvm::Align

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 23 05:30:21 PDT 2019


courbet accepted this revision.
courbet added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp:1993
   // Emit the function pointers in the target-specific order
-  const llvm::Align Align = llvm::Align(DL.getPointerPrefAlignment());
+  const llvm::Align Align = DL.getPointerPrefAlignment();
   llvm::stable_sort(Structors, [](const Structor &L, const Structor &R) {
----------------
move after the sort ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67914





More information about the llvm-commits mailing list