[PATCH] D52295: [DebugInfoMetadata] Added support to generate packed_decimal encoding related dwarf info.

[Paul Robinson via Phabricator via llvm-commits]

probinson added a comment.

In https://reviews.llvm.org/D52295#1251426, @Chirag wrote:

> Yes, with this patch, only decimal scale is supported and i will add binary scale as well. (will modify this patch), and related changes too.
>  and about the front-end, we have a pl1/cobol llvm based front-end (in development) which generates this info.


Awesome; is this the OpenVMS port or something else?  I had thought there were maybe two people in the world with both COBOL and LLVM experience.
COBOL doesn't need binary scale but I believe PL/I does (not that I've used PL/I in the past 25 years).  You'll want to think about whether it's simpler to use a single API for any scaled type, or split out decimal from binary.  Binary scaled obviously doesn't need most of the new parameters.


https://reviews.llvm.org/D52295