David Truby

Do we need _users_ to be able to use these for debugging? Because, as developers we could just build with -DLLVM_ENABLE_ASSERTIONS and test with that in release mode (which is...

Ah, I see what you mean now. I agree that there's a difference between `assert` and unreachable code, and also that there's a difference between something we want to use...

@dwblaikie thanks for your advice here! I am willing to commit myself to take a much deeper look at error handling as a first priority post-merge; but I think it'll...

@sscalpone at least according the to the C++ standard, NDEBUG does nothing other than enable the assert macro. What other behaviour are you expecting from it that makes it a...

Taskloop checks are now implemented except for branching in and out of a loop, which needs also doing for standard loops.

@ichoyjx Sorry for letting this rot for so long! I've added some more tests including the one you requested, and I've added back the restriction on reduction clauses on taskloop...

I haven't had a chance to test it yet. However, even if we wanted to stay with `std` classes `list` isn't the right choice here. For example in the parser...

Looks to me like in the Parser `std::list` is necessary with the current design, because pointers are taken into lists which are later spliced on to other lists. So while...

> > > Question: should the namespaces also need to be changed? > Like: > namespace Fortran::common { > namespace Fortran::decimal { In clang and llvm the namespace names are...

The f18 driver is intended to be temporary and a new driver based on the Clang driver code is currently being worked on. Perhaps we should keep an eye on...