[PATCH] D67091: Fix for headers having the same name as a directory
Alex Lorenz via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 12 11:47:51 PDT 2019
arphaman added a subscriber: jkorous.
arphaman added a comment.
In D67091#1667821 <https://reviews.llvm.org/D67091#1667821>, @kousikk wrote:
> Sorry about the delay on this - I was OOO (back now).
>
> 1. I added tests.
> 2. I couldn't add isDirectory() check to createFile() since that resulted in failures to normal scenarios where it was previously passing.
>
> PTAL!
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.
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).
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