MrTrillian added a comment.

In D154130#4487292 <https://reviews.llvm.org/D154130#4487292>, @jdenny wrote:

> 3. Extend lit's own test suite to cover it.

I submitted an update with your suggestions #1 and #2 but this #3 is much more 
difficult because of the nature of `%{t:real}`. I'm not sure I can have a 
source/target path that includes symlinks in a way that allows me to test this, 
and even more so with substitute drives since we can't know which drive letters 
are unallocated.


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

https://reviews.llvm.org/D154130

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

Reply via email to