=?utf-8?q?Félix?= Cloutier <[email protected]>,
=?utf-8?q?Félix?= Cloutier <[email protected]>,
=?utf-8?q?Félix?= Cloutier <[email protected]>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/[email protected]>
================
@@ -213,6 +213,16 @@ class CodeGenOptions : public CodeGenOptionsBase {
///< larger debug info than `Basic`.
};
+ enum class BoolFromMem {
+ Strict, ///< In-memory bool values are assumed to be 0 or 1, and any
other
+ ///< value is UB.
+ Truncate, ///< Convert in-memory bools to i1 by checking if the least
+ ///< significant bit is 1.
+ NonZero, ///< Convert in-memory bools to i1 by checking if any bit is set
+ ///< to 1.
----------------
apple-fcloutier wrote:
Something we currently use as in "the name already exists" or "the change
references it"? I think you may have been looking at an older version of the
change, this is now an `enum class`, so if the concern was scoping it should be
resolved. (With that said, within `CodeGenOptions`, there was no conflict.)
https://github.com/llvm/llvm-project/pull/160790
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
