- User Since
- Oct 10 2016, 10:44 AM (181 w, 1 d)
Thu, Mar 26
Sat, Mar 21
Wed, Mar 18
@modocache : thanks for the heads up, I'm investigating the issue.
Tue, Mar 17
Mon, Mar 16
Renamed registration constructor to use a more llvm-ish name.
@MaskRay no longer a WIP ;-)
@MaskRay gentle up :-)
Fri, Mar 13
Help on function escape in cgi:
Wed, Mar 11
@MaskRay are you ok with that version?
Tue, Mar 10
because that will cause duplicate registration of llvm::cl::opt flags.
Cleaned up code, using a few macros to avoid duplication.
Mon, Mar 9
@MaskRay The library version prooved to be a bad path: we wanted clang-cpp to always link to a shared library, which interacts in ungraceful ways with BUILD_SHARED_LIBS.
Thu, Mar 5
@grimar ok, change added to the commit.
Wed, Mar 4
This passes the following configurations:
Fix lto-test bugs + convert more files, still not ready for review.
Tue, Mar 3
Recommited as of 1454c27b60447d969d0c1ecaf20b2186fe9d85ec
Mon, Mar 2
For the record: test patched by b52355f8a196b5040dc2e42870bf8c459306cfaa
Feb 28 2020
Feb 27 2020
Update test case, according to @efriedma suggestion.
Add a test case that exhibits the overflow in previous implementation + minor nits from @hokein
@nhaehnle : something like
@lhames is that okay like this?
Feb 26 2020
Thanks for working on this! I ran into this once and manually undefined the macro, so the workaround looks decent. However, I share @hans concerns about the detection part. You most likely want to compile minimal C++ program using std::min (and maybe other functions) to detect if the actual standard library used matches our requirements.
Obsoleted by https://reviews.llvm.org/D74878
llvm-go is still a thing, this patch is no longer needed
Feb 25 2020
Fix include order
Use the <deque> approach
Nice work, LGTM too.
Feb 24 2020
Other copy/paste typo.
As a side note, pre-reserving the buffer size gives interesting speedup, see http://quick-bench.com/Lp6OwO2etW1YEmJayVWn3U8JDiY
Restore duplication, fix the bug in two separate locations.
@Meinersbur LGTM, thanks
Feb 22 2020
Feb 21 2020
As hinted by @efriedma , only remove lowering to *_finite calls in Codegen.
Add a configure feature test to filter out tests that depend on platform linker, as hinted by @MaskRay
Feb 20 2020
Other approach to the problem: modify Value::SetNameImpl
Finally, the test case :-)
Feb 19 2020
Added test case.
Feb 18 2020
Take @Meinersbur review into account.
I'll revert and propose an updated patch then.
Reverted through b8bea9346af4f2644c9a1bd29710c5e3efbbd7d3
Tested without standalone build with polly linked statically and dynamically with success.