<div dir="ltr">The trailing semicolon is missing in the usages of this macro in LLVM... I honestly don't know what's the preferred way here.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 19, 2014 at 7:12 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Wed, Mar 19, 2014 at 2:38 AM, Alexey Samsonov <<a href="mailto:samsonov@google.com">samsonov@google.com</a>> wrote:<br>

> Author: samsonov<br>
> Date: Wed Mar 19 04:38:31 2014<br>
> New Revision: 204219<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=204219&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=204219&view=rev</a><br>
> Log:<br>
> Remove extra semicolon for -Wpedantic<br>
<br>
</div>Could we modify the macro in such a way as to require the trailing<br>
semicolon instead? I believe that's the preferred way to have<br>
macros... (but I could be wrong)<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Modified:<br>
>     lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h<br>
><br>
> Modified: lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h?rev=204219&r1=204218&r2=204219&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h?rev=204219&r1=204218&r2=204219&view=diff</a><br>

> ==============================================================================<br>
> --- lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h (original)<br>
> +++ lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h Wed Mar 19 04:38:31 2014<br>
> @@ -101,7 +101,7 @@ typedef std::vector<Hex8> ContentBytes;<br>
>  typedef std::vector<uint32_t> IndirectSymbols;<br>
><br>
>  /// A typedef so that YAML I/O can encode/decode section attributes.<br>
> -LLVM_YAML_STRONG_TYPEDEF(uint32_t, SectionAttr);<br>
> +LLVM_YAML_STRONG_TYPEDEF(uint32_t, SectionAttr)<br>
><br>
>  /// Mach-O has a 32-bit and 64-bit section record.  This normalized form<br>
>  /// can support either kind.<br>
> @@ -122,10 +122,10 @@ struct Section {<br>
><br>
><br>
>  /// A typedef so that YAML I/O can encode/decode the scope bits of an nlist.<br>
> -LLVM_YAML_STRONG_TYPEDEF(uint8_t, SymbolScope);<br>
> +LLVM_YAML_STRONG_TYPEDEF(uint8_t, SymbolScope)<br>
><br>
>  /// A typedef so that YAML I/O can encode/decode the desc bits of an nlist.<br>
> -LLVM_YAML_STRONG_TYPEDEF(uint16_t, SymbolDesc);<br>
> +LLVM_YAML_STRONG_TYPEDEF(uint16_t, SymbolDesc)<br>
><br>
>  /// Mach-O has a 32-bit and 64-bit symbol table entry (nlist), and the symbol<br>
>  /// type and scope and mixed in the same n_type field.  This normalized form<br>
> @@ -142,7 +142,7 @@ struct Symbol {<br>
>  };<br>
><br>
>  /// A typedef so that YAML I/O can (de/en)code the protection bits of a segment.<br>
> -LLVM_YAML_STRONG_TYPEDEF(uint32_t, VMProtect);<br>
> +LLVM_YAML_STRONG_TYPEDEF(uint32_t, VMProtect)<br>
><br>
>  /// Segments are only used in normalized final linked images (not in relocatable<br>
>  /// object files). They specify how a range of the file is loaded.<br>
> @@ -179,7 +179,7 @@ struct BindLocation {<br>
>  };<br>
><br>
>  /// A typedef so that YAML I/O can encode/decode export flags.<br>
> -LLVM_YAML_STRONG_TYPEDEF(uint32_t, ExportFlags);<br>
> +LLVM_YAML_STRONG_TYPEDEF(uint32_t, ExportFlags)<br>
><br>
>  /// A normalized export entry.  Only used in normalized final linked images.<br>
>  struct Export {<br>
> @@ -193,7 +193,7 @@ struct Export {<br>
><br>
><br>
>  /// A typedef so that YAML I/O can encode/decode mach_header.flags.<br>
> -LLVM_YAML_STRONG_TYPEDEF(uint32_t, FileFlags);<br>
> +LLVM_YAML_STRONG_TYPEDEF(uint32_t, FileFlags)<br>
><br>
>  ///<br>
>  struct NormalizedFile {<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>
</div>