search

halide / Halide

a language for fast, portable data-parallel computation
https://halide-lang.org
Other
5.86k stars 1.07k forks source link

Remove Introspection #8273

Closed steven-johnson closed 3 months ago

steven-johnson commented 3 months ago

Introspection (to provide better error messages + automatic var/func/etc names) has always been kinda handy but kinda fragile, and with the evolution of the DWARF standard it's become broken for newer compilers. We don't have the bandwidth to fix it, and many large customers (e.g. Google) have never been able to rely on it, and given that it can cause crashes in some unusual situations (e.g. when embedded inside a Go app), it's time to say goodbye.

Alas! Poor Introspection. I knew him, Horatio. A feature of infinite jest, of most excellent fancy. It hath borne me on his back a thousand times.

steven-johnson commented 3 months ago

Finally green, PTAL

mcourteaux commented 2 months ago

I think we should update the tutorials to add explicit naming now, and teach how to use stmt and conceptual stmt files.