[PATCH] D100629: [Debug-Info][DBX] DW_AT_alignment should not be generated when dwarf version is not 5

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

As @probinson mentioned in https://reviews.llvm.org/D99250#2651953

> Our practice has been for a "tuning" option to, effectively, expand into other options to control various bits of behavior.  Basically, there should never be behavior that can be controlled _only_ via the tuning option.  This is a design decision that was debated at length when we introduced tuning.

(@probinson - do we have that written down somewhere?)

So this would have to be changed to add an LLVM flag (only has to be added in DwarfDebug.cpp, most likely - it doesn't have to be a published flag for Clang, and probably shouldn't be)

But this probably gets into Paul's suggestion that it might be easier to add one strict mode flag ( https://reviews.llvm.org/D100630#2694681 ) to group all this (I wouldn't mind adding that as a single flag in DwarfDebug.cpp for now - whether or not that gets plumbed up through into a CLang flag should probably be handled separately, but would be easy enough to do) under.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100629