[clang] [llvm] [clang-tools-extra] [llvm-objcopy] Add --gap-fill and --pad-to options (PR #65815)
Fangrui Song via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 6 22:00:39 PST 2023
================
@@ -738,6 +739,36 @@ objcopy::parseObjcopyOptions(ArrayRef<const char *> RawArgsArr,
if (auto Arg = InputArgs.getLastArg(OBJCOPY_extract_partition))
Config.ExtractPartition = Arg->getValue();
+ if (const auto *A = InputArgs.getLastArg(OBJCOPY_gap_fill)) {
+ if (Config.OutputFormat != FileFormat::Binary)
+ return createStringError(
+ errc::invalid_argument,
+ "'--gap-fill' is only supported for binary output");
+ ErrorOr<uint64_t> Val = getAsInteger<uint64_t>(A->getValue());
+ if (!Val)
+ return createStringError(Val.getError(), "--gap-fill: bad number: %s",
+ A->getValue());
+ uint8_t ByteVal = Val.get();
+ if (ByteVal != Val.get())
+ if (Error E = reportWarning(llvm::createStringError(
----------------
MaskRay wrote:
I see that objcopy uses a warning, but an error seems more appropriate and the difference between warning/error should not matter for the user.
https://github.com/llvm/llvm-project/pull/65815
More information about the cfe-commits
mailing list