[PATCH] D50524: [Hexagon] Replace fatal error with remark in HexagonISelLowering

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 9 12:54:45 PDT 2018


efriedma added a comment.

Converting the memory operation to an explicit trap seems overly complicated, as opposed to just fixing whatever isel pattern is assuming the immediate is divisible by 4.  But I don't know the Hexagon backend that well.

Could you use the optimization remark emission infrastructure for this, instead of defining your own DiagnosticInfo?  SelectionDAG::getORE() returns an optimization remark emitter.  Granted, that wouldn't do exactly the same thing, since optimization remarks are only emitted when requested.


Repository:
  rL LLVM

https://reviews.llvm.org/D50524





More information about the llvm-commits mailing list