[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