glsl: report no function instead of empty candidate list
authorErik Faye-Lund <erik.faye-lund@collabora.com>
Thu, 13 Jun 2019 10:03:27 +0000 (12:03 +0200)
committerErik Faye-Lund <erik.faye-lund@collabora.com>
Thu, 25 Jul 2019 15:20:10 +0000 (17:20 +0200)
commitc5f14322965cc006038e293902412e76ad268767
treeeb654f38e1edbeb699c288f971bb4ddfc951b2b4
parent7e1fe81f565fb72c63705b516d3e3b4c06184fc5
glsl: report no function instead of empty candidate list

When generating the error message for a missing function error where
all available overloads were missing due to a too low GLSL version, we
used to report something like this:

---8<---
0:224(14): error: no matching function for call to
           `textureCubeLod(samplerCube, vec3, float)'; candidates are:
0:224(14): error: type mismatch
---8<---

This is a pretty confusing error message, and can throw people off when
debugging. So let's instead check if any overload is available before we
decide what to print. This allow us to report something like this
instead:

---8<---
0:224(14): error: no function with name 'textureCubeLod'
0:224(14): error: type mismatch
---8<---

This is arguably easier to understand for programmers, and doesn't send
you on a wild goose chase to figure out what argument is wrong just
because you stopped reading the message prematurely. I'm of course
referring to a friend, not me. For sure. I would never do that.

Signed-off-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/compiler/glsl/ast_function.cpp