Unify unix and windows versions of FileManager::UniqueDirContainer and FileManager::UniqueFileContainer
Sean Silva
silvas at purdue.edu
Wed Jul 31 13:41:25 PDT 2013
On Wed, Jul 31, 2013 at 12:27 PM, Rafael EspĂndola <
rafael.espindola at gmail.com> wrote:
> On 31 July 2013 14:57, Sean Silva <silvas at purdue.edu> wrote:
> > - ::Emit32(Out, (uint32_t) StatBuf->st_ino);
> > - ::Emit32(Out, (uint32_t) StatBuf->st_dev);
> > - ::Emit16(Out, (uint16_t) StatBuf->st_mode);
> > - ::Emit64(Out, (uint64_t) StatBuf->st_mtime);
> > - ::Emit64(Out, (uint64_t) StatBuf->st_size);
> > - delete StatBuf;
> > + ::Emit64(Out, Data->UniqueID.getFile());
> > + ::Emit64(Out, Data->UniqueID.getDevice());
> > + ::Emit64(Out, Data->ModTime);
> > + ::Emit64(Out, Data->Size);
> > + delete Data;
> >
> > This seems like it changes the PTH file format. Is there some format
> version
> > number you need to bump somewhere? Or is this a "private" format?
>
> I was assuming it was private. I.E., the reader must be the same as
> the writer. Is that not the case?
>
idk; I just wanted to highlight a potential issue.
-- Sean Silva
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130731/969ae5b0/attachment.html>
More information about the cfe-commits
mailing list