[PATCH] D71732: Move the sysroot attribute from DIModule to DICompileUnit

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 9 14:56:21 PST 2020


aprantl added a comment.

> Some mysterious CFI failures I haven't looked into yet, but were pretty clearly blamed on this change & went away with a revert.

Because the flag was previously not (de)serialized and now shows up in LTO builds where it was dropped before?
Well I guess I can serialize false for now without preventing this from getting fixed.

Any opinions on this patch otherwise?


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

https://reviews.llvm.org/D71732





More information about the llvm-commits mailing list