[llvm] [Hexagon] Drop NodeAddr::operator<, fix bad asserts directives (PR #210192)

[Fateme Hosseini via llvm-commits]

================
@@ -358,8 +358,6 @@ template <typename T> struct NodeAddr {
     return !operator==(NA);
   }
 
-  bool operator<(const NodeAddr<T> &NA) const { return Id < NA.Id; }
----------------
fhossein-quic wrote:

Right, I dropped it to avoid touching the target-independent `RDFGraph.h` in my main XQF patch, and switched to using NodeId everywhere instead (basically inlining the lookup). But , then later reintroduced as part of the compliance checker patch, so now postRA is inconsistent, some spots key on NodeId, others on NodeAddr. If you'd rather keep operator<, I could refactor this patch to use NodeAddr consistently instead?

https://github.com/llvm/llvm-project/pull/210192