[llvm] [lit] Implement builtin umask (PR #94621)

Paul T Robinson via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 6 08:17:59 PDT 2024


================
@@ -565,6 +566,18 @@ class SHFILEOPSTRUCTW(Structure):
     return ShellCommandResult(cmd, "", stderr.getvalue(), exitCode, False)
 
 
+def executeBuiltinUmask(cmd, shenv):
+    """executeBuiltinUmask - Change the current umask."""
+    if len(cmd.args) != 2:
+        raise InternalShellError(cmd, "'umask' supports only one argument")
----------------
pogo59 wrote:

Consistency is good. I don't feel that strongly about it to request a separate comment-fix patch.

https://github.com/llvm/llvm-project/pull/94621


More information about the llvm-commits mailing list