[PATCH] D54043: Adding debug info to support Fortran (part 1)

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 27 14:32:11 PST 2018


probinson added a comment.

In D54043#1310486 <https://reviews.llvm.org/D54043#1310486>, @schweitz wrote:

> Thanks, Paul.
>
> Or this could go in first and supply additional tests for your new bit format. :)
>
> For what it's worth, the overloaded bit-encoding follows the precedent already established by LLVMDIFlagIndirectVirtualBase.


Sorry that it's holding you up, but if it's all the same to you, I'd rather put off the ecstasy of moving an existing flag from one word to the other.
I don't view DIFlagIndirectVirtualBase as a *good* precedent, either.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D54043





More information about the llvm-commits mailing list