[PATCH] D42082: Add DWARF for discriminated unions
    Tom Tromey via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Jan 16 11:46:46 PST 2018
    
    
  
tromey added a comment.
In https://reviews.llvm.org/D42082#977368, @probinson wrote:
> DWARF discriminated unions were likely designed to support Pascal variant records, which do allow multiple fields per variant.  If we're going to support discriminated unions (which seems reasonable) then it ought to be general enough to handle the Pascal case.
The reason I did not do this is that I didn't need it for Rust, and it seemed to me that it would be better for someone truly needing this functionality to add it -- I felt I would not do a good job guessing what might be needed in the future.
Repository:
  rL LLVM
https://reviews.llvm.org/D42082
    
    
More information about the llvm-commits
mailing list