[PATCH] D54884: [bugpoint] Find 'opt', etc., in bugpoint directory

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 28 08:30:20 PST 2018


davide added a comment.

As somebody who used bugpoint fairly extensively over the course of the years, I have to say I always considered a little error prone the fact that bugpoint looks up in the PATH to find `opt`.
So, I tend to be in favor of this patch, but I'm wondering whether it's worth to look in the $PATH instead of just looking in the current directory.
Realistically bugpoint is a tool for developers, so chances are you're going to run it directly from `bin` using the compiler/interpreter/assembler you just built.

You might consider asking on llvm-dev how people use the tool, and if there are no strong objections, we might be able to change the behavior. Other than that, the patch looks straightforward enough.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D54884





More information about the llvm-commits mailing list