[PATCH] D23766: DebugInfo: introduce DIFlagsUnderlying type for debug info flags

Victor via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 26 02:39:52 PDT 2016


vleschuk added a comment.

Changed underlying flags type name to match naming requirements. I think having additional typedef here is necessary that's the idea of the patch (also described in inline comment). As for moving DIFlags to DIBuilder: I don't think this is a good idea: this type logically belongs to DebugInfoMetadata (which is used by DIBuilder). Declaring DIFlags in DIBuilder will lead to cross-dependency.


https://reviews.llvm.org/D23766





More information about the llvm-commits mailing list