[PATCH] D127310: [clang][driver] fix to correctly set devtoolset on RHEL

Quinn Pham via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 10 07:42:46 PDT 2022


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



More information about the cfe-commits mailing list