[PATCH] D23766: DebugInfo: use strongly typed enum for debug info flags

Victor via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 30 14:29:15 PDT 2016


vleschuk added a comment.

In https://reviews.llvm.org/D23766#529553, @dblaikie wrote:

> Ah, I guess having DIFlags inside DINode helps scope the enumerators inside it (DINode::FlagZero rather than having "FlagZero" pollute the llvm namespace).


Correct. I also wanted to move DIFlags outside DINode, but than reconsidered it.


https://reviews.llvm.org/D23766





More information about the llvm-commits mailing list