Because this is not tge only part of ghe library search algorithm, and it’s not necessarily an error if the variable isn’t set
On Sun, Oct 22, 2017 at 1:56 PM Zhihao Yuan via Phabricator < revi...@reviews.llvm.org> wrote: > lichray marked an inline comment as done. > lichray added a comment. > > In https://reviews.llvm.org/D39162#903179, @zturner wrote: > > > Please don't throw an exception here. Instead, write this as: > > > > lit_config.warning('Unable to determine shared library path variable > for platform {}'.format(platform.system())) > > > > > > > Why a warning rather than fatal? > > > https://reviews.llvm.org/D39162 > > > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
