diff options
| author | Hans Wennborg <hans@hanshq.net> | 2025-09-30 10:23:04 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-09-30 08:23:04 +0000 |
| commit | 170b5fde8fc23525049ebbde8377f59971ee9b3f (patch) | |
| tree | 4cb029eedafa33dcaf4193ad11998509ee7cbb28 /clang/lib/Serialization/ModuleCache.cpp | |
| parent | 635910d14e3f4be1921c5c130cfe7aed7237c619 (diff) | |
| download | llvm-170b5fde8fc23525049ebbde8377f59971ee9b3f.zip llvm-170b5fde8fc23525049ebbde8377f59971ee9b3f.tar.gz llvm-170b5fde8fc23525049ebbde8377f59971ee9b3f.tar.bz2 | |
[Modules] Make -module-file-info print macro names in deterministic order (#161332)
Developers reported non-deterministic output from `-module-file-info`,
thinking this reflected non-determinism in the .pcm files themselves.
However, it turned out it was the printing that was non-deterministic:
```
$ cat /tmp/a.h
#define FOO 1
#define BAR 2
$ build/bin/clang -cc1 -std=c++20 -x c++ -emit-header-unit /tmp/a.h -o /tmp/a.pcm
$ build/bin/clang -cc1 -module-file-info /tmp/a.pcm | grep -A2 Definitions
Macro Definitions:
FOO
BAR
$ build/bin/clang -cc1 -module-file-info /tmp/a.pcm | grep -A2 Definitions
Macro Definitions:
BAR
FOO
```
Making the output deterministic also simplifies the test.
This is a follow-up to 360c5fe54c0758c73bf85453fd2913f371adc7d5
Diffstat (limited to 'clang/lib/Serialization/ModuleCache.cpp')
0 files changed, 0 insertions, 0 deletions