[PATCH] D67091: Fix for headers having the same name as a directory
Kousik Kumar via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 12 12:42:25 PDT 2019
kousikk added a comment.
> I was unable to reproduce the failures when I moved the check to createFile. What kind of failures did you see? Did you move it right to the start of the function? I would recommend rebasing the patch and retrying.
I saw an assertion error - but I realize now that it was because it was not at the top of the function. I had put it after `Entry->getContents()` which was throwing an assertion. I now realize that for `getContents()` to work, the entry needs to be a file. Thanks for correcting me!
> Also, could you please rename headerwithdirname.cpp to headerwithdirname_input.cpp when copying it to ensure FileCheck can match the filename without matching the temporary path (see https://reviews.llvm.org/D67379 for test fixes that @jkorous applied).
Done
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67091/new/
https://reviews.llvm.org/D67091
More information about the cfe-commits
mailing list