[PATCH] D121097: [C++20][Modules][HU 3/5] Emit module macros for header units.

Iain Sandoe via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 8 12:42:12 PST 2022


iains updated this revision to Diff 413913.
iains added a comment.

updated to remove a single-use variable.

Having some difficulty in reproducing the CI fails outside the harness, however the
code setting the removed variable could have faulted if there was no macro data for
the identifier.  Set the IsHeaderUnit variable explicitly.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121097/new/

https://reviews.llvm.org/D121097

Files:
  clang/include/clang/Serialization/ASTWriter.h
  clang/lib/Serialization/ASTWriter.cpp
  clang/test/Modules/cxx20-hu-04.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D121097.413913.patch
Type: text/x-patch
Size: 9606 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220308/dc80829a/attachment-0001.bin>


More information about the cfe-commits mailing list