[PATCH] D67433: [Alignment] Use llvm::Align in MachineFunction and TargetLowering - fixes mir parsing

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 11 03:01:20 PDT 2019


courbet added a comment.

Can you point to the tests that have anything interesting to look at ?



================
Comment at: llvm/lib/Target/ARC/ARCMachineFunctionInfo.h:37
         ReturnStackOffset(-1U), MaxCallStackReq(0) {
-    // Functions are 4-byte (2**2) aligned.
-    MF.setLogAlignment(2);
+    // Functions are 4-bytes aligned.
+    MF.setAlignment(llvm::Align(4));
----------------
no `s`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67433





More information about the llvm-commits mailing list