[lld] r204219 - Remove extra semicolon for -Wpedantic

Alexey Samsonov samsonov at google.com
Wed Mar 19 08:20:30 PDT 2014


The trailing semicolon is missing in the usages of this macro in LLVM... I
honestly don't know what's the preferred way here.


On Wed, Mar 19, 2014 at 7:12 PM, David Blaikie <dblaikie at gmail.com> wrote:

> On Wed, Mar 19, 2014 at 2:38 AM, Alexey Samsonov <samsonov at google.com>
> wrote:
> > Author: samsonov
> > Date: Wed Mar 19 04:38:31 2014
> > New Revision: 204219
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=204219&view=rev
> > Log:
> > Remove extra semicolon for -Wpedantic
>
> Could we modify the macro in such a way as to require the trailing
> semicolon instead? I believe that's the preferred way to have
> macros... (but I could be wrong)
>
> >
> > Modified:
> >     lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h
> >
> > Modified: lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h
> > URL:
> http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h?rev=204219&r1=204218&r2=204219&view=diff
> >
> ==============================================================================
> > --- lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h (original)
> > +++ lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFile.h Wed Mar 19
> 04:38:31 2014
> > @@ -101,7 +101,7 @@ typedef std::vector<Hex8> ContentBytes;
> >  typedef std::vector<uint32_t> IndirectSymbols;
> >
> >  /// A typedef so that YAML I/O can encode/decode section attributes.
> > -LLVM_YAML_STRONG_TYPEDEF(uint32_t, SectionAttr);
> > +LLVM_YAML_STRONG_TYPEDEF(uint32_t, SectionAttr)
> >
> >  /// Mach-O has a 32-bit and 64-bit section record.  This normalized form
> >  /// can support either kind.
> > @@ -122,10 +122,10 @@ struct Section {
> >
> >
> >  /// A typedef so that YAML I/O can encode/decode the scope bits of an
> nlist.
> > -LLVM_YAML_STRONG_TYPEDEF(uint8_t, SymbolScope);
> > +LLVM_YAML_STRONG_TYPEDEF(uint8_t, SymbolScope)
> >
> >  /// A typedef so that YAML I/O can encode/decode the desc bits of an
> nlist.
> > -LLVM_YAML_STRONG_TYPEDEF(uint16_t, SymbolDesc);
> > +LLVM_YAML_STRONG_TYPEDEF(uint16_t, SymbolDesc)
> >
> >  /// Mach-O has a 32-bit and 64-bit symbol table entry (nlist), and the
> symbol
> >  /// type and scope and mixed in the same n_type field.  This normalized
> form
> > @@ -142,7 +142,7 @@ struct Symbol {
> >  };
> >
> >  /// A typedef so that YAML I/O can (de/en)code the protection bits of a
> segment.
> > -LLVM_YAML_STRONG_TYPEDEF(uint32_t, VMProtect);
> > +LLVM_YAML_STRONG_TYPEDEF(uint32_t, VMProtect)
> >
> >  /// Segments are only used in normalized final linked images (not in
> relocatable
> >  /// object files). They specify how a range of the file is loaded.
> > @@ -179,7 +179,7 @@ struct BindLocation {
> >  };
> >
> >  /// A typedef so that YAML I/O can encode/decode export flags.
> > -LLVM_YAML_STRONG_TYPEDEF(uint32_t, ExportFlags);
> > +LLVM_YAML_STRONG_TYPEDEF(uint32_t, ExportFlags)
> >
> >  /// A normalized export entry.  Only used in normalized final linked
> images.
> >  struct Export {
> > @@ -193,7 +193,7 @@ struct Export {
> >
> >
> >  /// A typedef so that YAML I/O can encode/decode mach_header.flags.
> > -LLVM_YAML_STRONG_TYPEDEF(uint32_t, FileFlags);
> > +LLVM_YAML_STRONG_TYPEDEF(uint32_t, FileFlags)
> >
> >  ///
> >  struct NormalizedFile {
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>



-- 
Alexey Samsonov, MSK
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140319/cd2c9ed7/attachment.html>


More information about the llvm-commits mailing list