Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

compute_remainder_modulus analysis fails for cast<int>(some float) #2

Closed
abadams opened this issue Jul 31, 2012 · 1 comment
Closed
Labels

Comments

@abadams
Copy link
Member

abadams commented Jul 31, 2012

This was a bug listed on the trello board. It needs to be tested to see if it's still a bug and fixed if so.

jrk added a commit that referenced this issue Aug 2, 2012
Add Arg expression for Scalar arguments

- Add Arg expression to IR, codegen
- Add int32, int64 Scalar Arg unpacking to c wrapper function interface
- Pass dynamic width/height/channels as Scalar args in brightness example and cg_test_runner.cpp
@abadams
Copy link
Member Author

abadams commented Aug 5, 2012

Turns out it was fixed a while ago. Also added compute_remainder_modulus for Calls, which was failing.

@abadams abadams closed this as completed Aug 5, 2012
zvookin pushed a commit that referenced this issue May 26, 2016
dkurt referenced this issue in dkurt/Halide Sep 5, 2017
BGR2Gray Halide+OpenCV sample
slomp pushed a commit that referenced this issue Feb 26, 2018
dsharletg pushed a commit that referenced this issue May 8, 2018
steven-johnson pushed a commit that referenced this issue Jul 27, 2018
shoaibkamil pushed a commit that referenced this issue Oct 11, 2018
steven-johnson added a commit that referenced this issue Aug 19, 2020
Previous fix broke LLVM 11 (I was too eager to land, sorry)
steven-johnson added a commit that referenced this issue Aug 19, 2020
Previous fix broke LLVM 11 (I was too eager to land, sorry)
brent-carmer referenced this issue in InteonCo/Halide Feb 28, 2022
 Patch auto-scheduler the random seed for weight generation is configurable via the HL_RANDOM_WEIGHT_SEED env variable.
RootButcher referenced this issue in RootButcher/Halide-rustbinding-old Apr 21, 2022
RootButcher referenced this issue in RootButcher/Halide-rustbinding-old Apr 21, 2022
steven-johnson added a commit that referenced this issue Aug 16, 2022
steven-johnson added a commit that referenced this issue Aug 16, 2022
* Export HalidePythonExtensionHelpers.cmake for installs

* oops

* fixes

* Fix broken code in target_export_script()

* oops #2

* Add WITH_SOABI to stubs as well as AOT

* More fixes

* Update CMakePresets.json

* Update CMakePresets.json
alexreinking pushed a commit that referenced this issue Aug 16, 2022
* Export HalidePythonExtensionHelpers.cmake for installs

* oops

* fixes

* Fix broken code in target_export_script()

* oops #2

* Add WITH_SOABI to stubs as well as AOT

* More fixes

* Update CMakePresets.json

* Update CMakePresets.json
steven-johnson added a commit that referenced this issue Sep 23, 2022
This PR started out as a quick fix to add Python bindings for the `add_requirements` methods on Pipeline and Generator (which were missing), but expanded a bit to fix other issues as well:
- The implementation of `Generator::add_requirement` was subtly wrong, in that it only worked if you called the method after everything else in your `generate()` method. Now we accumulate requirements and insert them at the end, so you can call the method anywhere.
- We had C++ methods that took both an explicit `vector<Expr>` and also a variadic-template version, but the former required a mutable vector... and fixing this to not require that ended up creating ambiguity about which overloaded call to use. Added an ugly enable_if thing to resolve this.

(Side note #1: overloading methods to have both templated and non-templated versions with the same name is probably something to avoid in the future.)

(Side note #2: we should probably thing more carefully about using variadic templates in our public API in the future; we currently use it pretty heavily, but it tends to be messy and hard to reason about IMHO.)
steven-johnson added a commit that referenced this issue Sep 23, 2022
* add_requirement() maintenance

This PR started out as a quick fix to add Python bindings for the `add_requirements` methods on Pipeline and Generator (which were missing), but expanded a bit to fix other issues as well:
- The implementation of `Generator::add_requirement` was subtly wrong, in that it only worked if you called the method after everything else in your `generate()` method. Now we accumulate requirements and insert them at the end, so you can call the method anywhere.
- We had C++ methods that took both an explicit `vector<Expr>` and also a variadic-template version, but the former required a mutable vector... and fixing this to not require that ended up creating ambiguity about which overloaded call to use. Added an ugly enable_if thing to resolve this.

(Side note #1: overloading methods to have both templated and non-templated versions with the same name is probably something to avoid in the future.)

(Side note #2: we should probably thing more carefully about using variadic templates in our public API in the future; we currently use it pretty heavily, but it tends to be messy and hard to reason about IMHO.)

* tidy

* remove underscores
steven-johnson added a commit that referenced this issue Jul 18, 2023
Some sneakiness going on.
steven-johnson added a commit that referenced this issue Jul 19, 2023
* Fix float16 under asan, attempt #2

Some sneakiness going on.

* Update float16_t.cpp
vksnk added a commit that referenced this issue Nov 6, 2023
ardier pushed a commit to ardier/Halide-mutation that referenced this issue Mar 3, 2024
* Export HalidePythonExtensionHelpers.cmake for installs

* oops

* fixes

* Fix broken code in target_export_script()

* oops halide#2

* Add WITH_SOABI to stubs as well as AOT

* More fixes

* Update CMakePresets.json

* Update CMakePresets.json
ardier pushed a commit to ardier/Halide-mutation that referenced this issue Mar 3, 2024
* add_requirement() maintenance

This PR started out as a quick fix to add Python bindings for the `add_requirements` methods on Pipeline and Generator (which were missing), but expanded a bit to fix other issues as well:
- The implementation of `Generator::add_requirement` was subtly wrong, in that it only worked if you called the method after everything else in your `generate()` method. Now we accumulate requirements and insert them at the end, so you can call the method anywhere.
- We had C++ methods that took both an explicit `vector<Expr>` and also a variadic-template version, but the former required a mutable vector... and fixing this to not require that ended up creating ambiguity about which overloaded call to use. Added an ugly enable_if thing to resolve this.

(Side note halide#1: overloading methods to have both templated and non-templated versions with the same name is probably something to avoid in the future.)

(Side note halide#2: we should probably thing more carefully about using variadic templates in our public API in the future; we currently use it pretty heavily, but it tends to be messy and hard to reason about IMHO.)

* tidy

* remove underscores
ardier pushed a commit to ardier/Halide-mutation that referenced this issue Mar 3, 2024
* Fix float16 under asan, attempt halide#2

Some sneakiness going on.

* Update float16_t.cpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant