From 06b2192aa2bb6698c863388c5d085ba5b1f28374 Mon Sep 17 00:00:00 2001 From: Tim Foley Date: Mon, 9 Oct 2017 13:38:33 -0700 Subject: Preprocessor: fix `undef` and redefinition (#204) * Preprocessor: fix `undef` and redefinition The logic for `undef` directives was failing to suppress macro expansion when reading the name to un-define, and so it wasn't actually working at all. We didn't notice this because we didn't have a test case, and users hadn't tried it. The logic for `define` had a similar bug, which meant that any attempt to define an already-defined macro would fail with a cryptic error, rather than raising the intended warning. Test cases have been added for both issues, along with the fixes. * fixup: add expected output for tests added --- tests/preprocessor/define-redefine.slang | 10 ++++++++++ 1 file changed, 10 insertions(+) create mode 100644 tests/preprocessor/define-redefine.slang (limited to 'tests/preprocessor/define-redefine.slang') diff --git a/tests/preprocessor/define-redefine.slang b/tests/preprocessor/define-redefine.slang new file mode 100644 index 000000000..42b7a4dbe --- /dev/null +++ b/tests/preprocessor/define-redefine.slang @@ -0,0 +1,10 @@ +//TEST:SIMPLE: + +// Confirm that we handle redefinition +// of an existing define (with a diagnostic) + +#define FOO 1.0f + +float foo() { return FOO + 2.0; } + +#define FOO 2.0f -- cgit v1.2.3