>
> bool Program::buildFromUnit(const ir::Unit &unit, std::string &error) {
> @@ -158,6 +161,13 @@ namespace gbe {
> for (const auto &pair : set) {
> const std::string &name = pair.first;
> Kernel *kernel = this->compileKernel(unit,
> name, !OCL_STRICT_CONFORMANCE);
> + if (!kernel) {
> + error += name;
> + error += ":(GBE): error: failed in Gen backend.\n";
> + if (OCL_OUTPUT_BUILD_LOG)
> + llvm::errs() << error;
> + return false;
> + }
I think we can output the errCode in the error log. That would be more
meaningful. But the patch now is acceptable, we can add it later.
And the whole patchset LGTM.
Thanks!
Ruiling
_______________________________________________
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet
