[PATCH] D65945: [LLVM][Alignment] Make functions using log of alignment explicit

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 30 08:08:34 PDT 2019


gchatelet added a subscriber: arphaman.
gchatelet added a comment.

In D65945#1643692 <https://reviews.llvm.org/D65945#1643692>, @gchatelet wrote:

> @arphaman do you mind having a look at the MIR parser / writer parts?
>  How is alignment supposed to be serialized for MIR? As power of two or a log of alignment?


Upon closer inspection, it seems like the PowerOfTwo requirement comes from https://github.com/llvm/llvm-project/commit/547a83b4ebd1
@arsenm do you mind having a look ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65945





More information about the llvm-commits mailing list