carlosgalvezp wrote: > It's ok as far as it works. Personally if we do not utilize that optional, > then probably it should be removed.
It's essentially a revert. I agree that our usage of optional in the Options is not good, we should improve that on a separate patch. https://github.com/llvm/llvm-project/pull/134215 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
