quinnp added a comment.

Thanks @tbaeder, @MaskRay, and @nemanjai.

> I did not remove that on purpose, so adding it back makes sense to me.

I've updated the patch to remove the path without the `/root/usr` suffix and 
exclusively add the path with the `root/usr` suffix.

> This change also needs a unit test.

I've updated the existing test case to match this change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127310

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

Reply via email to