[all-commits] [llvm/llvm-project] c2c259: const char* for LLVMTargetMachineEmitToFile's argu...

deadalnix via All-commits all-commits at lists.llvm.org
Sat May 7 07:41:07 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c2c259224bb30b089206dd69dc44aefddffec2f4
      https://github.com/llvm/llvm-project/commit/c2c259224bb30b089206dd69dc44aefddffec2f4
  Author: Amaury Séchet <deadalnix at gmail.com>
  Date:   2022-05-07 (Sat, 07 May 2022)

  Changed paths:
    M llvm/include/llvm-c/TargetMachine.h
    M llvm/lib/Target/TargetMachineC.cpp

  Log Message:
  -----------
  const char* for LLVMTargetMachineEmitToFile's argument

The `LLVMTargetMachineEmitToFile` takes a `char* Filename` right now, but it doesn't modify it.
This is annoying to use in the case where you want to pass a const string, because you either have to remove the const, or copy it somewhere else and pass that. Either way, it's not very nice.

I added a const and clang formatted it. This shouldn't break any ABI in my opinion.
I'm sorry but I didn't know whom to put as reviewer for this, so I chose someone with a lot of commits from the .cpp file.

Reviewed By: deadalnix

Differential Revision: https://reviews.llvm.org/D124453




More information about the All-commits mailing list