Vicente Mataix =?utf-8?q?Ferrándiz?= <vmat...@altair.com>,
Vicente Mataix =?utf-8?q?Ferrándiz?= <vmat...@altair.com>,
Vicente Mataix =?utf-8?q?Ferrándiz?= <vmat...@altair.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/127...@github.com>
================
@@ -446,6 +448,12 @@ async def main() -> None:
action="store_true",
help="Allow empty enabled checks.",
)
+ parser.add_argument(
+ "-ignore-insert-conflict",
----------------
carlosgalvezp wrote:
This option is currently a no-op as it is, because it doesn't change anything.
The default value is True, and this option can only set it to True. If
anything, there should be a `-no-ignore-insert-conflict` option that sets it to
False, given a default of True.
I still don't fully understand what problem this patch is trying to solve,
could you ellaborate? Maybe this patch is not needed at all?
https://github.com/llvm/llvm-project/pull/127066
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
