[PATCH] D25620: DebugInfo: introduce DIAlignment type
Adrian Prantl via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 14 09:59:26 PDT 2016
> On Oct 14, 2016, at 9:48 AM, David Blaikie <dblaikie at gmail.com> wrote:
>
> Could someone point me to where the discussion for adding this type came out of? I didn't spot it at a cursory glance of the previous/existing threads.
>
This came out of the review thread in:
D25073: [DebugInfo]: preparation to implement DW_AT_alignment
https://reviews.llvm.org/D25073
where I argued that we should not be using a full uint64_t for the alignment fields in the DI.* metadata nodes. I don't think the concrete solution of introducing a new alignment type has been discussed here before.
-- adrian
More information about the llvm-commits
mailing list