benlangmuir added a comment.
@fmayer thanks, fix is: https://reviews.llvm.org/D131017
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130934/new/
https://reviews.llvm.org/D130934
___
cfe-commits mailing
fmayer added a comment.
This CL seems to have an MSAN violation:
https://lab.llvm.org/buildbot/#/builders/74/builds/12529/steps/11/logs/stdio
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130934/new/
https://reviews.llvm.org/D130934
This revision was automatically updated to reflect the committed changes.
Closed by commit rGb4c6dc2e6637: [clang] Update code that assumes
FileEntry::getName is absolute NFC (authored by benlangmuir).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
bnbarham accepted this revision.
bnbarham added a comment.
This revision is now accepted and ready to land.
LGTM, thanks for doing this
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130934/new/
https://reviews.llvm.org/D130934
benlangmuir created this revision.
benlangmuir added reviewers: bnbarham, jansvoboda11.
Herald added a project: All.
benlangmuir requested review of this revision.
Herald added projects: clang, clang-tools-extra.
Herald added a subscriber: cfe-commits.
It's an accident that we started return