gchatelet added a comment.

In D97223#2581587 <https://reviews.llvm.org/D97223#2581587>, @jyknight wrote:
> In D97223#2581126 <https://reviews.llvm.org/D97223#2581126>, @rjmccall wrote:
>
>> Looks like you didn't update the .bc reader/writer and the .ll 
>> printer/parser.
>
> The instructions already had alignment support added to the type constructor 
> and IR/BC in prior changes by gchatelet -- this is just updating the 
> IRBuilder constructor.

For the record the previous patches introducing the alignment were D87443 
<https://reviews.llvm.org/D87443> and D83465 <https://reviews.llvm.org/D83465>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97223

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to