Steven Johnson comments

Results 261 comments of


Steven Johnson

Fix #5175: Add an extra check for input dependencies of the async producer

LGTM but @zvookin should probably review

Fix #5175: Add an extra check for input dependencies of the async producer

Is this still just awating review?

Fix #5175: Add an extra check for input dependencies of the async producer

Is this just waiting on review?

Fix #5175: Add an extra check for input dependencies of the async producer

Where does this PR stand?

Fix #5175: Add an extra check for input dependencies of the async producer

This PR is pretty old. Is it still active?

Disable LLVM loop optimization by default (Issue #4113)

> Or perhaps the only people using this flag are insiders anyway. This is what I'd suspect. But maybe announcing this on halide-dev@ and Gitter to get more input would...

Disable LLVM loop optimization by default (Issue #4113)

Hm, maybe this really is moving the needle in this case: ``` cd bin/build/tmp ; /home/halidenightly/build_bot/worker/linux-64-gcc7-testbranch-trunk/halide-Release/bin/performance_boundary_conditions unbounded : 88274.299000 us constant_exterior : 1574301.813000 us Error: constant_exterior is 17.834204 times slower...

Disable LLVM loop optimization by default (Issue #4113)

Bringing this back up to see if we (finally) want to land this for Halide 13.

Disable LLVM loop optimization by default (Issue #4113)

Since we're about to cut a branch for Halide 12, I propose we land this for Halide 13, since it's a potentially-breaking change.

Disable LLVM loop optimization by default (Issue #4113)

> Sounds good, but I believe there are one or two things to merge into master before we have a complete 12.0 branch. Yep, just want to get it reviewed...