[PATCH] D66490: [NewPM] Enable the New Pass Manager by Default in Clang

2020-04-03 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan added a comment. > It seems that we can go ahead with this patch if there's no other objections. > I reran a `ninja check-clang` with this patch and all tests seem to pass (at > least when running on x64-linux). Actually disregard this. It seems that there are some compiler-rt

[PATCH] D66490: [NewPM] Enable the New Pass Manager by Default in Clang

2020-04-03 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan added a comment. In D66490#1958747 , @aeubanks wrote: > In D66490#1638708 , @hfinkel wrote: > > > In D66490#1638162 , @rupprecht > > wrote: > > > > > We already

[PATCH] D66490: [NewPM] Enable the New Pass Manager by Default in Clang

2020-04-02 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks added a comment. In D66490#1638708 , @hfinkel wrote: > In D66490#1638162 , @rupprecht wrote: > > > We already know that we don't want this enabled for tsan builds due to > >

[PATCH] D66490: [NewPM] Enable the New Pass Manager by Default in Clang

2019-08-20 Thread Hal Finkel via Phabricator via cfe-commits
hfinkel added a comment. In D66490#1638162 , @rupprecht wrote: > We already know that we don't want this enabled for tsan builds due to > https://bugs.llvm.org/show_bug.cgi?id=42877, but I don't even know if anyone > else will hit it (it's only when

[PATCH] D66490: [NewPM] Enable the New Pass Manager by Default in Clang

2019-08-20 Thread Jordan Rupprecht via Phabricator via cfe-commits
rupprecht added a comment. We already know that we don't want this enabled for tsan builds due to https://bugs.llvm.org/show_bug.cgi?id=42877, but I don't even know if anyone else will hit it (it's only when building one particular library). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D66490: [NewPM] Enable the New Pass Manager by Default in Clang

2019-08-20 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan added a comment. As of now, from running `check-llvm` and `check-clang` there's 2 failing tests, but I'm trying to get those addressed. - `Clang :: CodeGen/split-lto-unit.c` which should be resolved by D66488 - `Clang ::

[PATCH] D66490: [NewPM] Enable the New Pass Manager by Default in Clang

2019-08-20 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan created this revision. leonardchan added reviewers: chandlerc, echristo, hfinkel, lattner, rupprecht. leonardchan added a project: clang. Herald added subscribers: dexonsmith, mehdi_amini, mgorny. The new PM serves as a replacement for the legacy PM, and promises better codegen,