keith added a comment.

I'm realizing more now that these changes break the original intent of 
rG0be9ca7c0f9a733f846bb6bc4e8e36d46b518162 
<https://reviews.llvm.org/rG0be9ca7c0f9a733f846bb6bc4e8e36d46b518162> as well, 
given that it seems like the core canonicalization of the path before passing 
it down was the intent, but that's also the part that appears to be causing the 
issue. @JDevlieghere I would definitely appreciate some advice here, I'm still 
trying to see if I can satisfy the new test case without breaking the tests you 
changed, but I think they might fundamentally be at odds


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109128/new/

https://reviews.llvm.org/D109128

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to