[PATCH] D11958: Add a -gmodules option to the clang driver.

Adrian Prantl via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 20 14:37:20 PDT 2015


aprantl added a comment.

Maybe this is missing the point, but a hash collision between module files with different formats should be highly unlikely because we add the module format into the hash.

  llvm::hash_code Hash =
    llvm::hash_combine(DirName.lower(), FileName.lower(),
                       HSOpts->ModuleFormat);

Or do you mean a collision in spite of this?

- adrian


Repository:
  rL LLVM

http://reviews.llvm.org/D11958





More information about the cfe-commits mailing list