[PATCH] D110019: [gn build] improve write_cmake_config to be truthy and exception friendly

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 20 10:40:16 PDT 2021


thakis added a comment.

Can you explain the motivation behind this?

> Besides, in order to substiute #cmakedefine var with value,
> it needs to access "values[key]" which will raises a "KeyError"
> exception if searching for a key that does not exist.

That part is intentional, to make sure the list of keys passed to the script matches exactly the list of keys needed in the cmake file.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110019/new/

https://reviews.llvm.org/D110019



More information about the llvm-commits mailing list