This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG223e99fb698d: [clang][deps] Fix module context hash for
constant strings (authored by benlangmuir).
Changed prior to commit:
https://reviews.llvm.
Bigcheese accepted this revision.
Bigcheese added a comment.
Thanks for fixing this. LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143027/new/
https://reviews.llvm.org/D143027
___
cfe-commits maili
jansvoboda11 accepted this revision.
jansvoboda11 added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143027/new/
https://reviews.llvm.org/D143027
___
benlangmuir created this revision.
benlangmuir added reviewers: jansvoboda11, Bigcheese.
Herald added a project: All.
benlangmuir requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
We were not hashing constant strings in the command-line, only
