sas added a comment.

In https://reviews.llvm.org/D40537#937196, @zturner wrote:

> You could use llvm's range adapters to make this slightly better.
>
>   auto Bytes = makeArrayRef(m_uuid, m_num_uuid_bytes);
>   return llvm::find(Bytes, 0) != Bytes.end();
>   
>
> Another option would just be `return *this != UUID(m_num_uuid_bytes);`


We want at least one **non-zero** element, we don't want a zero-element, so the 
proposed code wouldn't work. I'm not sure if there's an llvm utility that 
allows doing that directly without having to pass a lambda.


https://reviews.llvm.org/D40537



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to