[PATCH] Simplify DWARFCompileUnit / DWARFTypeUnit constructors (NFC).

Frederic Riss friss at apple.com
Wed Oct 8 10:41:14 PDT 2014


>>! In D5659#9, @dblaikie wrote
:> If the problem is just the number of arguments - we can wrap that in a struct and achieve the same terseness without pushing the choice down into layers that are otherwise agnostic of that detail.

It's not only about the number of arguments, I like the fact that the Unit choses the sections it needs. I find it more logical from an OO POV than having a helper in DWARFUnitSection make the choice. Ideally, I'd like to get rid of parseDWO totally and have the DWOness be a Unit type attribute (that was the point of my comment regarding the type hierarchy). I have the feeling that this commit gets us one step closer to that.

But I can see your point though, won't argue if you reject the change.

http://reviews.llvm.org/D5659






More information about the llvm-commits mailing list