<div dir="ltr">Joerg,<div><br></div><div>Are you talking about the "non-default #pragma pack value might change the alignment of struct or union members in the included file" warning? If yes, that commit got reverted (before 5.0 branch I think). I'm working on a fixed commit that would avoid this particular warning.</div><div><br></div><div>Alex</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 21 July 2017 at 11:48, Joerg Sonnenberger via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Jan 13, 2016 at 04:36:15AM -0000, Xinliang David Li via llvm-commits wrote:<br>
> Modified: llvm/trunk/include/llvm/<wbr>ProfileData/CoverageMapping.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ProfileData/CoverageMapping.h?rev=257571&r1=257570&r2=257571&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/llvm/trunk/include/<wbr>llvm/ProfileData/<wbr>CoverageMapping.h?rev=257571&<wbr>r1=257570&r2=257571&view=diff</a><br>
> ==============================<wbr>==============================<wbr>==================<br>
> --- llvm/trunk/include/llvm/<wbr>ProfileData/CoverageMapping.h (original)<br>
> +++ llvm/trunk/include/llvm/<wbr>ProfileData/CoverageMapping.h Tue Jan 12 22:36:15 2016<br>
</span><span class="">> @@ -479,14 +489,50 @@ inline std::error_code make_error_code(c<br>
>  // [Encoded Region Mapping Data]<br>
>  LLVM_PACKED_START<br>
>  template <class IntPtrT> struct CovMapFunctionRecord {<br>
> -  #define COVMAP_FUNC_RECORD(Type, LLVMType, Name, Init) Type Name;<br>
> -  #include "llvm/ProfileData/<wbr>InstrProfData.inc"<br>
> +#define COVMAP_FUNC_RECORD(Type, LLVMType, Name, Init) Type Name;<br>
> +#include "llvm/ProfileData/<wbr>InstrProfData.inc"<br>
<br>
</span>This part now triggers a warning that the default packed attribute via<br>
LLVM_PACKED_START affects the types in the included file. Is that<br>
intentional? This applies to the 5.0 branch as well.<br>
<br>
Joerg<br>
<div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>