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

[Santanu Das 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; }
----------------
quic-santdas wrote:

As far as I remember, I had to put this to create a map in the postRAQFPHandle pass, where the key is a NodeAddr<T> type. Insertion/loopkup for a keytype requires the operator< since the map is implemented as a red-black tree, which requires less-than comparison.

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