[PATCH] D50246: [RISCV] Add support for computing sysroot for riscv32-unknown-elf
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 10 11:03:33 PDT 2018
lebedev.ri added a comment.
In https://reviews.llvm.org/D50246#1229177, @kristina wrote:
> In https://reviews.llvm.org/D50246#1229152, @rogfer01 wrote:
>
> > Hi @kristina .
> >
> > Sure, I didn't mean to do that broader change here. Apologies if it read that way.
> >
> > Would it be acceptable to add an empty `--sysroot=` to the test? I can post the change for review in another diff.
> >
> > Thanks a lot.
>
>
> Yes, you can submit another diff for this specific test, just use "Update Diff" and add yours on top which should open it up for re-review or use "Commandeer Revision" (and then submit another diff since it was previously closed). Up to your judgement.
Please don't perform necromancy on already committed and closed differentials (unless the commit was reverted, of course).
Do open new differentials.
Repository:
rC Clang
https://reviews.llvm.org/D50246
More information about the cfe-commits
mailing list