[PATCH] D156330: [hexagon] restore library path arguments

Brian Cain via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 26 14:20:57 PDT 2023


bcain added a comment.

In D156330#4535974 <https://reviews.llvm.org/D156330#4535974>, @MaskRay wrote:

> Tip: use `%clang -fdriver-only -Werror -v` to test that a command produces no warning or error and has an exit code of 0. Without it your `Args.ClaimAllArgs(options::OPT_L);` change is untested.

This test fails on the baseline because of the `CHECK-NOT: warning:` IIRC.  But your suggestion sounds like a good one anyways, will do!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156330



More information about the cfe-commits mailing list