[PATCH] D146215: [BOLT] Reject symbols pointing to section end

Vladislav Khmelevsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 17 12:28:56 PDT 2023


yota9 added a comment.

LGTM, but small nit -  I would swap this new condition with the " if (!Section->isText()) " condition above, since this one is more "univesal" and might appear not inly in text section. And although currently it does not really matter it probably better from the logical standpoint.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146215



More information about the llvm-commits mailing list