[PATCH] D84602: [MSP430] Expose msp430_builtin calling convention to C code

Anatoly Trosinenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 14 03:19:06 PDT 2020


atrosinenko added a comment.

In D84602#2206512 <https://reviews.llvm.org/D84602#2206512>, @aaron.ballman wrote:

> I still think this case needs some work to keep the bots happy. I would probably go with:
>
> case CC_MSP430Builtin:
>
>   // FIXME: Currently unsupported
>   break;
>
> But perhaps @echristo has opinions since this involves debugging information.

Agree, unless there exist some straightforward way to add a new vendor-specific extension to `include/llvm/BinaryFormat/Dwarf.def`. But I suspect there would be some standardization stuff and not sure it will actually be that useful.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84602/new/

https://reviews.llvm.org/D84602



More information about the llvm-commits mailing list