jingham added a comment.

In D71372#1782549 <https://reviews.llvm.org/D71372#1782549>, @clayborg wrote:

> In D71372#1782538 <https://reviews.llvm.org/D71372#1782538>, @jingham wrote:
>
> > In D71372#1782536 <https://reviews.llvm.org/D71372#1782536>, @clayborg 
> > wrote:
> >
> > > I believe it is ok for permissions to succeed as long as they don't 
> > > return invalid permissions. For any address outside any mapped ranges, we 
> > > should have zero as the permissions. Looking up address mappings for zero 
> > > will return
> > >
> > > [0x00000000 - 0x100000000) ---
> > >
> > > no permisssions are represented as "---". Then you can take the end 
> > > address and look it up, and continue. So as long as we aren't getting 
> > > bogus values back, we are good.
> >
> >
> > Then what does the bool return mean?
>
>
> If there is no memory map info in the process plug-in at all, then false will 
> be returned.


That means to answer the question "did GetLoadAddressPermissions return valid 
permissions for load address 0xABCD" I have to check both the return value and 
if any of the permission values are unknown?  That seems like an awkward API.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71372



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to