[PATCH] D86362: Allow graph writer to render DOT nodes using HTML.

[Madhur Amilkanthwar via Phabricator via llvm-commits]

madhur13490 added a comment.

I can't access the link in the description of the change. Moreover, I don't understand the purpose of making several calls to `DTraits.renderNodeUsingHTML(Node)`. In its current form, the code queries just once and uses the value throughout the code, with your change you'd be calling several times. If this is a pre-requisite to some other change then I'd like to see that change so understand the whole picture. (Phabricator allows you to link revisions and even create a dependency FWIW.)


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

https://reviews.llvm.org/D86362