[PATCH] D45243: [XRay][clang] Consolidate runtime and link-time flag processing (NFC)

Dean Michael Berris via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 4 05:22:35 PDT 2018


dberris added a comment.

In https://reviews.llvm.org/D45243#1056549, @devnexen wrote:

> Do the unit tests have same outcome with these changes ? Otherwise, appreciate the simplification.


Yes, locally, `ninja check-all` on Linux works just fine for me. No idea whether this will be the case for the buildbots, but that's easy enough to address when I get LGTM for the patch (and it lands, preferably daytime in Sydney).


https://reviews.llvm.org/D45243





More information about the cfe-commits mailing list