DhruvSrivastavaX wrote:

> Do you actually need to parse the headers yourself? The reason for using the 
> llvm implementation was so that we can avoid that. FWICS, all of this data 
> should be available through the `(file|auxiliary)Header(32|64)` APIs.

Okay, I'll take a look at that. 



> BTW, this is exactly the kind of granularity I was looking for.

Happy to hear that! Will follow the similar practice.

> I don't know, but I wouldn't be worried about that. The test failure is more 
> of a concern.

Yeah, I think I missed the test case modification accordingly. Will update!

https://github.com/llvm/llvm-project/pull/116338
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to